verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 66 Releases Wiki Activity

refs #6397 emailValidation #1941

Merged
carlossa merged 13 commits from 6397-emailValidation into dev 2024-02-01 17:45:54 +00:00
Conversation 0 Commits 13 Files Changed 3 +8 -6
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 7cb7c7dace - Show all commits

7
loopback/locale/es.json
View File

@ -72,7 +72,7 @@
"The secret can't be blank": "La contraseña no puede estar en blanco",
"We weren't able to send this SMS": "No hemos podido enviar el SMS",
"This client can't be invoiced": "Este cliente no puede ser facturado",
"You must provide the correction information to generate a corrective invoice": "Debes informar la información de corrección para generar una factura rectificativa",
"You must provide the correction information to generate a corrective invoice": "Debes informar la información de corrección para generar una factura rectificativa",
"This ticket can't be invoiced": "Este ticket no puede ser facturado",
"You cannot add or modify services to an invoiced ticket": "No puedes añadir o modificar servicios a un ticket facturado",
"This ticket can not be modified": "Este ticket no puede ser modificado",
@ -336,5 +336,6 @@
"Incorrect pin": "Pin incorrecto.",
"You already have the mailAlias": "Ya tienes este alias de correo",
"The alias cant be modified": "Este alias de correo no puede ser modificado",
"No tickets to invoice": "No hay tickets para facturar"
}
"No tickets to invoice": "No hay tickets para facturar",
"An email is necessary": "An email is necessary"
carlossa marked this conversation as resolved Outdated
alexm commented 2024-01-18 13:53:37 +00:00
Outdated
Review
Copy Link

Falta traducir

Falta traducir
alexm commented 2024-01-22 07:03:34 +00:00
Outdated
Review
Copy Link

Falta traducir

Falta traducir
}

7
modules/client/back/methods/client/createWithUser.js
View File

@ -1,4 +1,4 @@
const {error} = require('fancy-log');
const UserError = require('vn-loopback/util/user-error');
module.exports = function(Self) {
Self.remoteMethod('createWithUser', {
@ -31,15 +31,16 @@ module.exports = function(Self) {
myOptions.transaction = tx;
}
let user;
jgallego commented 2024-02-02 08:27:19 +00:00
Review
Copy Link

@alexm valorares la opcio de posar-ho en back/models/vn-user.js com a una validacio nativa?

@alexm valorares la opcio de posar-ho en back/models/vn-user.js com a una validacio nativa?
alexm commented 2024-02-02 08:30:36 +00:00
Review
Copy Link

Per mi si, pero hi hauria que mirar els nulls que hi hajen com se comporta la restriccio, pq segurament qualsevol canvi en el model fallara

Per mi si, pero hi hauria que mirar els nulls que hi hajen com se comporta la restriccio, pq segurament qualsevol canvi en el model fallara
if (data.email) {
carlossa marked this conversation as resolved Outdated
alexm commented 2024-01-18 14:03:10 +00:00
Outdated
Review
Copy Link

Esto se puede refactorizar y dejar sin if else

Esto se puede refactorizar y dejar sin if else
const firstEmail = data.email ? data.email.split(',')[0] : null;
carlossa marked this conversation as resolved Outdated
alexm commented 2024-01-18 14:03:38 +00:00
Outdated
Review
Copy Link

Aqui como este codigo siempre va a tener data.email se puede quitar el hacer ? : (if else)

Aqui como este codigo siempre va a tener data.email se puede quitar el hacer ? : (if else)
const user = {
user = {
name: data.userName,
email: firstEmail,
password: String(Math.random() * 100000000000000)
};
} else
throw error;
throw new UserError(`An email is necessary`);
try {
const account = await models.VnUser.create(user, myOptions);