Javier Segarra

jsegarra commented on pull request verdnatura/salix-front#112

2023-11-24 13:04:08 +00:00

refs #6062 feat(arrayData): support exprBuilder

si coinciden todas las claves con el valor de props, quizas se puede hacer

jsegarra commented on pull request verdnatura/salix-front#112

2023-11-24 13:04:08 +00:00

refs #6062 feat(arrayData): support exprBuilder

Duda, esto es para eliminar los parámetros de búsqueda que tengan valor vacío o null

jsegarra commented on pull request verdnatura/salix-front#112

2023-11-24 13:04:08 +00:00

refs #6062 feat(arrayData): support exprBuilder

Lo miramos, porque este método se puede simplificar

jsegarra suggested changes for verdnatura/salix-front#112

2023-11-24 13:04:08 +00:00

refs #6062 feat(arrayData): support exprBuilder

jsegarra pushed to 6434-improve-signInLog at verdnatura/salix

2023-11-24 11:45:40 +00:00

c890dfeb7d Merge branch 'dev' of https://gitea.verdnatura.es/verdnatura/salix into 6434-improve-signInLog

cbcc2ba17c refs #6434 perf: remove default value param

74dd64126a Merge pull request '#5835 getTotals fixed' (!1854) from 5835-fixGetTotals into dev

b3584d9db5 Merge branch 'dev' of https://gitea.verdnatura.es/verdnatura/salix into 5835-fixGetTotals

4a62588464 Merge pull request '6028_route_getRouteByWorker' (!1857) from 6028_route_getRouteByWorker into dev

Compare 131 commits »

jsegarra pushed to 5858-zone_Collisions at verdnatura/salix

2023-11-24 11:30:55 +00:00

4383ebd7f1 refs #5858 feat: email notification

5d156e81ca refs #5858 perf: move files to current version folder

a2329e1276 refs #5858 feat: update template notification

57b8b69ae9 Merge branch 'dev' of https://gitea.verdnatura.es/verdnatura/salix into 5858-zone_Collisions

31c1add02b Merge pull request 'refactor: refs #4502 Deleted zoneEstimatedDelivery' (!1844) from 4502-zoneEstimatedDelivery into dev

Compare 16 commits »

jsegarra approved verdnatura/salix-front#110

2023-11-24 07:52:06 +00:00

refs #5835 migrateInvoiceIn

jsegarra approved verdnatura/salix-front#106

2023-11-24 07:37:43 +00:00

6157-actionClaim

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Revisamos linea 65!

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Lo revisamos!!

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Y esto también!!

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Lo que le pasas a notify podría ser una constante, no? Porque lo hacemos en la línea 148, 166, 195

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Podemos definir este valor como constante?

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

Si, los labels cambian, tienes que venir hasta el HTML?

jsegarra commented on pull request verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

esto no se usa, no?

jsegarra approved verdnatura/salix-front#106

2023-11-24 07:37:31 +00:00

6157-actionClaim

jsegarra commented on pull request verdnatura/salix-front#110

2023-11-22 13:58:18 +00:00

refs #5835 migrateInvoiceIn

Incluso te diría que en vez de tantos ifs, haría un objeto de validaciones, donde la key es el campo y el value es el mensaje a mostrar.

jsegarra commented on pull request verdnatura/salix-front#110

2023-11-22 13:50:39 +00:00

refs #5835 migrateInvoiceIn

Yo movería la lógica de create y save a funciones constantes porque estás duplicando código que hacen lo mismo

jsegarra commented on pull request verdnatura/salix-front#110

2023-11-22 13:50:39 +00:00

refs #5835 migrateInvoiceIn

lo miramos

jsegarra pushed to 5858-zone_Collisions at verdnatura/salix

2023-11-22 11:48:52 +00:00

b453abdb3a refs #5858 feat: new event in vn