#6274 workerTimeControl #1858
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1858
Loading…
Reference in New Issue
No description provided.
Delete Branch "6274-loginWorkerTimeControl"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
WIP: #6274 workerTimeControlto #6274 workerTimeControl@ -0,0 +1,35 @@
module.exports = Self => {
Self.remoteMethod('clockIn', {
veo que existe un archivo muy similar que llama al mismo procedimiento, lo has tenido en cuenta? conviene juntarlo en uno?
modules/worker/back/methods/worker-time-control/addTimeEntry.js
Ya se ha implementado, tenemos que mirarlo juntos conforme hemos hablado por rocket.
Refactor aplicado tras revisión en persona.
@alexm tras hablar con Javi, hemos visto que el método addTimeEntry llamaba a resendWeeklyHourEmail, ¿Es necesario? y, además, se comprobaba si es subordinado y si mismo pero, en es subordinado ya se comprueba que sea el mismo. Comentamos cuando vuelvas de vacaciones.
Tras mirarlo con Alex, se ha decidido que no se use weeklyHourRecordEmail cada vez que se ficha. No tiene sentido.
@ -0,0 +1,14 @@
INSERT INTO `account`.`role` (name, description)
Cambiar version a la ultima
@ -0,0 +36,4 @@
Object.assign(myOptions, options);
const query = 'CALL vn.workerTimeControl_clockIn(?, ?, ?)';
const [response] = await Self.rawSql(query, [workerFk, timed, direction], myOptions);
Si es el caso puedes hacer:
Es decir, hacer
const [[response]] =
y te evitas luego hacer response[0]@ -0,0 +29,4 @@
const query = `CALL vn.workerTimeControl_login(?)`;
const user = await Self.rawSql(query, [pin], myOptions);
if (!user) throw new UserError('Indique el pin.');
Falta traduccion
@ -0,0 +9,4 @@
('WorkerTimeControl', 'login', 'READ', 'ALLOW', 'ROLE', 'timeControl'),
('WorkerTimeControl', 'getClockIn', 'READ', 'ALLOW', 'ROLE', 'timeControl'),
('WorkerTimeControl', 'clockIn', 'WRITE', 'ALLOW', 'ROLE', 'timeControl'),
('VnUser', 'renewToken', 'WRITE', 'ALLOW', 'ROLE', 'timeControl');
el acl sobre renewToken ya debería estar definido para $owner, si no es asi definirlo
@juan podemos mirarlo?
New commits pushed, approval review dismissed automatically according to repository settings
Siguen habiendo archivos sql que no cuadran con la version actual
@alexm a mi no me sale ningún archivo para mover. Si me los puedes decir por rocket...
Los cambios realizados en renewToken han sido hechos con Juan @alexm , @jgallego
Dejar donde estaba el archivo db/changes/240201/00-updateCourtesyTime.sql
db/changes/234601/00-updateCourtesyTime.sql → db/changes/240201/00-updateCourtesyTime.sql
@ -0,0 +3,4 @@
describe('workerTimeControl login()', () => {
it('should correctly login', async() => {
const response = await models.WorkerTimeControl.login(9, {});
No hace falta pasar el objeto
New commits pushed, approval review dismissed automatically according to repository settings
lo que has quitado de timeEntry.spec porque es?
Porque el método que se está evaluando es clockIn, que es el que puede fallar. Solo he dejado los que tienen que ver con el Rol.
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
@ -331,6 +331,7 @@
"Cannot past travels with entries": "No se pueden pasar envíos con entradas",
"It was not able to remove the next expeditions:": "No se pudo eliminar las siguientes expediciones: {{expeditions}}",
"This user does not have an assigned tablet": "Este usuario no tiene tablet asignada",
"Incorrect pin.": "Pin incorrecto.",
ayer no hablamos de no poner puntos? son estos casos?
cierto
New commits pushed, approval review dismissed automatically according to repository settings