verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

4320-notificationQueue #1087

Merged
alexm merged 15 commits from 4320-notificationQueue into dev 2022-10-28 06:06:04 +00:00
Conversation 0 Commits 15 Files Changed 15 +480 -2
alexm commented 2022-10-14 11:28:19 +00:00
Member
Copy Link
No description provided.
alexm added the
CR / Tests passed
 label 2022-10-14 11:28:19 +00:00
alexm added 11 commits 2022-10-14 11:28:21 +00:00
gitea/salix/pipeline/head This commit is unstable Details
7895decf4e feat(notification): create models
gitea/salix/pipeline/head This commit looks good Details
9b48cbb078 feat(notification): send
gitea/salix/pipeline/head There was a failure building this commit Details
2b19688ca4 quit focus
gitea/salix/pipeline/head This commit looks good Details
24cb8e00bb creates notification sql in changes
gitea/salix/pipeline/head This commit looks good Details
c637f2c879 feat(send): use vn-print libary and adapt test
joan requested review from juan 2022-10-17 08:16:13 +00:00
juan requested changes 2022-10-17 08:24:25 +00:00
back/methods/notification/send.js
@ -0,0 +17,4 @@
Self.send = async options => {
if (process.env.NODE_ENV == 'test')
throw new UserError(`Action not allowed on the test environment`);
juan commented 2022-10-17 08:21:33 +00:00
Owner
Copy Link

En test no tiene que lanzar error y debe ejecutar todo el código a excepción de la linea que envía el correo, así se puede comprobar que el proceso funciona correctamente.

En test no tiene que lanzar error y debe ejecutar todo el código a excepción de la linea que envía el correo, así se puede comprobar que el proceso funciona correctamente.
alexm marked this conversation as resolved
alexm added 1 commit 2022-10-17 10:12:23 +00:00
gitea/salix/pipeline/head This commit looks good Details
aef47d46b1 env test
alexm added 1 commit 2022-10-17 10:12:53 +00:00
alexm added 1 commit 2022-10-17 10:14:23 +00:00
gitea/salix/pipeline/head This commit looks good Details
c24b975f26 remove node_env
alexm requested review from juan 2022-10-17 10:15:00 +00:00
juan approved these changes 2022-10-27 13:53:10 +00:00
alexm added 64 commits 2022-10-28 05:34:10 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
d30065f90a Add vn-input-account tag and logic
gitea/salix/pipeline/head There was a failure building this commit Details
c2813e3633 Removed console logs and other debugging elements
gitea/salix/pipeline/head There was a failure building this commit Details
dff378632b Changed the logic
gitea/salix/pipeline/head There was a failure building this commit Details
7ac7189a8a added maxlength binding, finished logic on insert
gitea/salix/pipeline/head There was a failure building this commit Details
796988bb18 Removed e.stopPropagation
gitea/salix/pipeline/head There was a failure building this commit Details
5ca39c9dd3 Modified TextField to have a insertable option
gitea/salix/pipeline/head This commit looks good Details
9c7ec6fdc9 Implemented Insertable in the textfield class
gitea/salix/pipeline/head This commit looks good Details
012cd68c89 Added menu to create a new supplier, not working
gitea/salix/pipeline/head There was a failure building this commit Details
e81c39d8dc Send delivery note to refund tickets
gitea/salix/pipeline/head This commit looks good Details
dc6c8f924d feat: Create new Supplier
gitea/salix/pipeline/head There was a failure building this commit Details
594cee12f3 add checks to the existence of object variables
gitea/salix/pipeline/head This commit looks good Details
1603619d81 removed debugging elements
gitea/salix/pipeline/head This commit looks good Details
0cdee89ac3 Button removed when user does not have permissions
gitea/salix/pipeline/head This commit looks good Details
f0b1591b14 Remove debugging elements
gitea/salix/pipeline/head This commit looks good Details
dc53aac163 Merge branch 'dev' into 2612-create-supplier
gitea/salix/pipeline/head There was a failure building this commit Details
cfdcf91be2 Merge branch 'dev' into 3505-closure_refund
gitea/salix/pipeline/head This commit looks good Details
c212dfed88 removed package-lock
gitea/salix/pipeline/head This commit looks good Details
cf50c4690a Merge branch 'dev' into 4153-supplier.fiscal-data
gitea/salix/pipeline/head There was a failure building this commit Details
70c9c3d788 initial back test created, modified DB statetments
gitea/salix/pipeline/head This commit looks good Details
e6481af3a9 delete: transaction
gitea/salix/pipeline/head There was a failure building this commit Details
9290cd6fb6 Finished back tests
gitea/salix/pipeline/head This commit looks good Details
047076f189 changed tests
gitea/salix/pipeline/head There was a failure building this commit Details
a204c6e2b0 Fixed the suppliers not appearing
gitea/salix/pipeline/head There was a failure building this commit Details
470b4485d5 Fixed test and DB structure
gitea/salix/pipeline/head There was a failure building this commit Details
db12ceb369 removed async from the redirect function
gitea/salix/pipeline/head There was a failure building this commit Details
b36acee458 Merge pull request '4153-supplier.fiscal-data' (!1068) from 4153-supplier.fiscal-data into dev 
Reviewed-on: #1068
Reviewed-by: Juan Ferrer <juan@verdnatura.es>
gitea/salix/pipeline/head Build queued... Details
73a69051a8 Merge branch 'dev' into 2612-create-supplier
gitea/salix/pipeline/head There was a failure building this commit Details
dcd858c7d5 feat(print): 4690 - Puppeteer cluster
gitea/salix/pipeline/head There was a failure building this commit Details
192e61a70d hotFix(route_index): fix driverRoutePdf
gitea/salix/pipeline/head There was a failure building this commit Details
7d502c6f57 Merge pull request 'Send delivery note to refund tickets' (!1086) from 3505-closure_refund into dev 
Reviewed-on: #1086
gitea/salix/pipeline/head This commit looks good Details
aa13034ee6 Merge branch 'dev' into 4153-supplier.fiscal-data
gitea/salix/pipeline/head This commit looks good Details
3564cbe246 Merge pull request 'MASTER_4689-hotFix(route_index): fix driverRoutePdf' (!1095) from hotFix_driverRoutePdf2 into master 
Reviewed-on: #1095
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head There was a failure building this commit Details
c23d58da69 fix if not ticket
gitea/salix/pipeline/head There was a failure building this commit Details
cf0b750304 Merge branch 'test' of https://gitea.verdnatura.es/verdnatura/salix into dev
gitea/salix/pipeline/head Build queued... Details
ceeab73cf2 Merge pull request '4091-dms_clean_fix2' (!1091) from 4091-dms_clean_fix2 into dev 
Reviewed-on: #1091
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head There was a failure building this commit Details
04be661b68 Merge pull request 'feat(print): 4690 - Puppeteer cluster' (!1094) from 4690-puppeteer_cluster into dev 
Reviewed-on: #1094
gitea/salix/pipeline/head This commit looks good Details
25e934a591 Merge branch 'dev' into 4153-supplier.fiscal-data
gitea/salix/pipeline/head There was a failure building this commit Details
6a17a634d4 hotFix(receipt): add receiptPdf
gitea/salix/pipeline/head This commit looks good Details
a66c5e6798 acl
gitea/salix/pipeline/head This commit looks good Details
fe95c45c42 Merge pull request 'MASTER_hotFix(receipt): add receiptPdf' (!1100) from hotFix_receiptPdf into master 
Reviewed-on: #1100
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head This commit looks good Details
e80754f280 Added new SQL version
gitea/salix/pipeline/head There was a failure building this commit Details
931a891f71 fix: check client.email exists
gitea/salix/pipeline/head There was a failure building this commit Details
4520443734 fix: Intl polyfill for spanish locale
gitea/salix/pipeline/head This commit looks good Details
5811a0f56e Merge pull request 'MASTER_hotfix: check if email exists' (!1103) from 4708-client.basic-data into master 
Reviewed-on: #1103
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head There was a failure building this commit Details
9089ba7346 requested changes
gitea/salix/pipeline/head This commit looks good Details
d60f6769b4 Merge branch 'dev' into 4153-supplier.fiscal-data
gitea/salix/pipeline/head This commit looks good Details
39b70ed75f Merge pull request '4153-supplier.fiscal-data' (!1096) from 4153-supplier.fiscal-data into dev 
Reviewed-on: #1096
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head This commit looks good Details
330af56d2f Intl polyfill
gitea/salix/pipeline/head There was a failure building this commit Details
76d248131c Merge branch 'dev' into 2612-create-supplier
gitea/salix/pipeline/head There was a failure building this commit Details
1196125350 Merge branch 'test' of https://gitea.verdnatura.es/verdnatura/salix into dev
gitea/salix/pipeline/head There was a failure building this commit Details
7b04464848 Merge branch 'dev' into 2612-create-supplier
gitea/salix/pipeline/head There was a failure building this commit Details
296a743d8a Fix test, remove console log
gitea/salix/pipeline/head There was a failure building this commit Details
418ba65c2e Merge pull request '2612-create-supplier' (!1089) from 2612-create-supplier into dev 
Reviewed-on: #1089
Reviewed-by: Joan Sanchez <joan@verdnatura.es>
gitea/salix/pipeline/head There was a failure building this commit Details
4334907d91 fix(database): updated sql files
gitea/salix/pipeline/head There was a failure building this commit Details
6a2bc5b593 Disabled back tests
gitea/salix/pipeline/head There was a failure building this commit Details
51f95f0261 Fixed error
gitea/salix/pipeline/head There was a failure building this commit Details
714fe32552 Fix error
gitea/salix/pipeline/head This commit looks good Details
80617dd578 Merge branch 'dev' into 4320-notificationQueue
vicent approved these changes 2022-10-28 06:05:25 +00:00
vicent left a comment
Contributor
Copy Link

Este PR ya había sido aprobado, solo se ha actualizado la rama.

Este PR ya había sido aprobado, solo se ha actualizado la rama.
alexm merged commit 88dda4523f into dev 2022-10-28 06:06:04 +00:00
alexm deleted branch 4320-notificationQueue 2022-10-28 06:06:04 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
juan
vicent
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#1087
No description provided.
Delete Branch "4320-notificationQueue"

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?