Tickets advance #462

Merged
jsegarra merged 6 commits from :feature/TicketsAdvance into dev 2024-06-25 12:23:41 +00:00
Contributor
No description provided.
hyervoni added 2 commits 2024-06-19 22:47:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
45cf4498d3
Cancel button change
hyervoni requested review from alexm 2024-06-19 22:48:07 +00:00
hyervoni requested review from jsegarra 2024-06-19 22:48:08 +00:00
wbuezas added 1 commit 2024-06-20 10:57:49 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0ba415c0ea
small changes
jsegarra requested changes 2024-06-20 21:51:26 +00:00
Dismissed
jsegarra left a comment
Member

(NO ES ERROR) 1. En ingles, los modales no muestran el numero de items seleccionados. En español si. He visto en Salix, y está idéntico a lo que ha hecho @wbuezas. Por lo tanto, lo arreglamos nosotros.
2. Las cabeceras de las columnas, diria que los textos están invertidos, porque la definición de las columnas está OK
3. Cuando le das a adelantar sin negativos, no te aparece el numero totoal de registros.
4. Cuando adelantas sin negativos, el mensaje del toast no está en español
5. El icono de adelantar sin negativos, no es el correcto. El correcto es icon="alt_route"

✅(NO ES ERROR) 1. En ingles, los modales no muestran el numero de items seleccionados. En español si. He visto en Salix, y está idéntico a lo que ha hecho @wbuezas. Por lo tanto, lo arreglamos nosotros. ✅2. Las cabeceras de las columnas, diria que los textos están invertidos, porque la definición de las columnas está OK ✅3. Cuando le das a adelantar sin negativos, no te aparece el numero totoal de registros. ✅4. Cuando adelantas sin negativos, el mensaje del toast no está en español ✅5. El icono de adelantar sin negativos, no es el correcto. El correcto es icon="alt_route"
jsegarra added 1 commit 2024-06-20 22:03:54 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0faa663169
fix: comments
jsegarra added 1 commit 2024-06-20 22:07:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
159b386d2c
fix: comment 4
jsegarra approved these changes 2024-06-25 11:03:31 +00:00
Dismissed
alexm approved these changes 2024-06-25 12:11:33 +00:00
Dismissed
jsegarra added 1 commit 2024-06-25 12:22:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
5685d99cd6
Merge branch 'dev' into feature/TicketsAdvance
jsegarra dismissed jsegarra’s review 2024-06-25 12:22:05 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jsegarra dismissed alexm’s review 2024-06-25 12:22:05 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jsegarra scheduled this pull request to auto merge when all checks succeed 2024-06-25 12:22:23 +00:00
jsegarra requested review from jsegarra 2024-06-25 12:23:28 +00:00
jsegarra approved these changes 2024-06-25 12:23:32 +00:00
jsegarra merged commit 1a79f733f0 into dev 2024-06-25 12:23:41 +00:00
jsegarra deleted branch feature/TicketsAdvance 2024-06-25 12:23:41 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 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-front#462
No description provided.