6275-silex-to-salix #1878

Merged
sergiodt merged 12 commits from 6275-silex-to-salix into dev 2023-12-14 05:52:08 +00:00
Member
No description provided.
sergiodt added 6 commits 2023-12-05 14:43:47 +00:00
gitea/salix/pipeline/head This commit looks good Details
3f7663da97
feat: refs #6275 crea back etExpeditionSummary
gitea/salix/pipeline/head This commit looks good Details
a0a9e299d9
feat: refs #6275 getTickets with phone
gitea/salix/pipeline/head This commit looks good Details
aa0794e136
Merge branch 'dev' into 6275-silex-to-salix
gitea/salix/pipeline/head This commit looks good Details
4071c06b5f
feat: refs #6275 getTickets con los campos de silex
gitea/salix/pipeline/head This commit looks good Details
318f7e3c0d
REFS #6275 feat:Silex_to_Salix
gitea/salix/pipeline/head This commit looks good Details
3692021b12
Merge branch 'dev' into 6275-silex-to-salix
sergiodt requested review from jgallego 2023-12-05 14:43:54 +00:00
jgallego requested changes 2023-12-07 08:11:49 +00:00
@ -0,0 +8,4 @@
IF (NEW.ticketFk IS NOT NULL)
THEN
UPDATE address
SET longitude = NEW.longitude,
Owner

si son null les volem guardar també?

si son null les volem guardar també?
Author
Member

Modificat per a que no es guarde

Modificat per a que no es guarde
@ -0,0 +26,4 @@
FOR EACH ROW
BEGIN
IF (NEW.longitude <> OLD.longitude OR NEW.latitude <> OLD.latitude OR NEW.ticketFk <> OLD.ticketFk)
Owner

hacer pruebas con valores NULL

hacer pruebas con valores NULL
Author
Member

Modificat

Modificat
@ -0,0 +1,64 @@
module.exports = Self => {
Self.remoteMethod('getExpeditionSummary', {
Owner

si ja va a dev faltaria el test de back

si ja va a dev faltaria el test de back
Author
Member

Pujat

Pujat
sergiodt added 1 commit 2023-12-07 10:40:01 +00:00
gitea/salix/pipeline/head This commit looks good Details
61ed53bd1d
REFS #6275 feat:Silex_to_Salix
sergiodt added 1 commit 2023-12-07 10:41:11 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
fa65cc84a9
REFS #6275 feat:Silex_to_Salix
sergiodt requested review from jgallego 2023-12-07 10:46:27 +00:00
sergiodt added 1 commit 2023-12-07 11:00:53 +00:00
gitea/salix/pipeline/head This commit looks good Details
9937fbed5e
REFS #6275 feat:Silex_to_Salix
jgallego requested changes 2023-12-07 11:28:43 +00:00
@ -0,0 +5,4 @@
it('should return a summary of expeditions for a route', async() => {
const result = await app.models.Route.getExpeditionSummary(routeId);
expect(result.length).toEqual(3);
Owner

busca un test que te un map en el que es recorre que tots els registres tenen la routeId = 3 així el test es dinamic

busca un test que te un map en el que es recorre que tots els registres tenen la routeId = 3 així el test es dinamic
Author
Member

Modificat test

Modificat test
sergiodt added 1 commit 2023-12-12 14:50:28 +00:00
gitea/salix/pipeline/head This commit looks good Details
e22201c840
Merge branch 'dev' into 6275-silex-to-salix
sergiodt added 1 commit 2023-12-13 06:26:10 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
435819cf3c
refs #6275 feat:Silex_to_Salix test getExpeditionSummary
sergiodt requested review from jgallego 2023-12-13 06:31:22 +00:00
jgallego approved these changes 2023-12-13 14:29:23 +00:00
sergiodt added 1 commit 2023-12-13 15:12:42 +00:00
gitea/salix/pipeline/head This commit looks good Details
af83519ee6
Merge branch 'dev' into 6275-silex-to-salix
sergiodt merged commit 0130130cee into dev 2023-12-14 05:52:08 +00:00
sergiodt deleted branch 6275-silex-to-salix 2023-12-14 05:52:09 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#1878
No description provided.