Javier Segarra

jsegarra created branch 6611_popup-proxy_descriptor in verdnatura/salix-front

2024-01-04 09:13:09 +00:00

jsegarra pushed to 6611_popup-proxy_descriptor at verdnatura/salix-front

2024-01-04 09:13:09 +00:00

jsegarra pushed to 5666-loggable_to_mixin at verdnatura/salix

2024-01-04 08:08:50 +00:00

f7d83b15bd Merge branch 'dev' into 5666-loggable_to_mixin

2a65ad6b31 refs #6635 build: new version

23b544c1ec refs #6635 fix: add translation

f93416e8c1 Merge pull request '#5914 - hotFix-transferInvoice' (!1893) from 5914-hotFix-transferInvoice into test

227afbe39d refs #5914 fix(invoiceOut_descriptorMenu): transalation

Compare 21 commits »

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

2024-01-04 08:05:00 +00:00

branch-PR-2

Las traducciones tienes * para marcar que son requeridos.

jsegarra suggested changes for verdnatura/salix-front#135

2024-01-04 08:05:00 +00:00

branch-PR-2

jsegarra approved verdnatura/salix#1893

2024-01-04 07:11:45 +00:00

#5914 - hotFix-transferInvoice

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Es worder o worker?

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Si de prop solo se va a sacar row, quizás, seria interesante hacer event: ({row})

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

2024-01-04 06:26:14 +00:00

branch-PR-2

El formato de este computed no es igual al resto.

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Los métodos de la 172 y 177, hacen lo mismo.

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Queda redundante hacer params: {id: id}, haciendo params:{id} sería suficiente

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Si props no se usa mas abajo, seria interesante desestructurarlo para eliminar las referencias a "prop."

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

2024-01-04 06:26:14 +00:00

branch-PR-2

Este componente de tipo listado solo tiene definido un botón, no hay una tabla como si ocurre en otros componentes del mismo tipo, es correcto?

jsegarra suggested changes for verdnatura/salix-front#135

2024-01-04 06:26:14 +00:00

branch-PR-2

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

2024-01-03 19:25:00 +00:00

branch-PR-2

La clase close-icon se ha perdido, o los valores que tenia están dentro del HTML en vez de definirlos en el CSS.

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

2024-01-03 19:25:00 +00:00

branch-PR-2

El cb de map, se puede simplificar

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

2024-01-03 19:25:00 +00:00

branch-PR-2

Si solo se usan 2 propiedades, desestructurar col

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

2024-01-03 19:25:00 +00:00

branch-PR-2

length es un numero y queda mas natural que sea > en vez de !==. No puede ser null/undefined porque se asume que siempre viene length

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

2024-01-03 19:25:00 +00:00

branch-PR-2

Si solo se usan 2 propiedades, desestructurar node

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

2024-01-03 19:25:00 +00:00

branch-PR-2

Traducciones literales en código