verdnatura/salix-front
verdnatura
/
salix-front
2
2
Fork 1
Code Issues Pull Requests 99 Releases Wiki Activity

feature/FaltantesModuloTravel #185

Merged
jsegarra merged 17 commits from :feature/FaltantesModuloTravel into dev 2024-02-09 13:11:23 +00:00
Conversation 1 Commits 17 Files Changed 23 +1387 -353
hyervoni commented 2024-02-06 18:14:00 +00:00
Contributor
Copy Link

https://redmine.verdnatura.es/issues/6812

https://redmine.verdnatura.es/issues/6812
hyervoni added 33 commits 2024-02-06 18:14:01 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6c8e736b17 changes in entry searchbar
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
584511bc62 Change EntryBuys table image proportions
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7089202fe1 remove comment
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
5280a4fa2d Remove '' in click.stop handle
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
2de02723ba Abstract repetitive returns in a function
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
ddcbc5d8d7 make EditTableCellForm componente reusable
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
67e2130374 Adjust spacing in entry create form
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
cde41ab337 Entry notes validation
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
28790705ea Sort entry observation options
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
0525f5fdd8 Add ItemDescriptor to EntryBuys table
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
5da5bb2047 get input events improvement
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
86bbc7cfd6 label input changes
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
02c163c5e0 small change
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
a7e653cbce Stop propagation on item descriptor cell click
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
f6b0404cb4 multiple changes
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
6773a9a696 Entry buys import open file explorer when clicking clip icon
gitea/salix-front/pipeline/pr-dev This commit looks good Details
225d600b52 resolve conflicts
gitea/salix-front/pipeline/pr-dev This commit looks good Details
018f251f3d small improvements
jsegarra requested review from alexm 2024-02-07 06:39:53 +00:00
jsegarra requested review from jsegarra 2024-02-07 06:39:53 +00:00
alexm requested changes 2024-02-07 08:28:25 +00:00
alexm left a comment
Member
Copy Link

/list falta chips en F. Envio y F.Entrega
/list al darle a clonar las fechas son un dia anterior al real
/descriptor crear entrada en /entry/create en Envio sale el numero en vez del string
/summary el campo entradas totales no lo muestra biem (Ver travel 3)
En http://localhost:9000/#/travel/3/thermographs/index al darle a crear salta error por:

read properties of undefined (reading 'id')
/list falta chips en F. Envio y F.Entrega /list al darle a clonar las fechas son un dia anterior al real /descriptor crear entrada en /entry/create en Envio sale el numero en vez del string /summary el campo entradas totales no lo muestra biem (Ver travel 3) En http://localhost:9000/#/travel/3/thermographs/index al darle a crear salta error por: ``` read properties of undefined (reading 'id') ```
wbuezas added 1 commit 2024-02-07 18:02:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f59c2409f1 travels corrections and improvements
wbuezas commented 2024-02-07 18:03:55 +00:00
Contributor
Copy Link

/list falta chips en F. Envio y F.Entrega
/list al darle a clonar las fechas son un dia anterior al real
/descriptor crear entrada en /entry/create en Envio sale el numero en vez del string
/summary el campo entradas totales no lo muestra biem (Ver travel 3)
En http://localhost:9000/#/travel/3/thermographs/index al darle a crear salta error por:

read properties of undefined (reading 'id')

@alexm todos los puntos corregidos!

Commit: f59c2409f1

> /list falta chips en F. Envio y F.Entrega > /list al darle a clonar las fechas son un dia anterior al real > /descriptor crear entrada en /entry/create en Envio sale el numero en vez del string > /summary el campo entradas totales no lo muestra biem (Ver travel 3) > En http://localhost:9000/#/travel/3/thermographs/index al darle a crear salta error por: > ``` > read properties of undefined (reading 'id') > ``` @alexm todos los puntos corregidos! Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/f59c2409f19631aa6716fbd554d6f48e9ad62856
jsegarra requested changes 2024-02-08 07:21:09 +00:00
src/pages/Travel/Card/TravelThermographsForm.vue
@ -0,0 +1,325 @@
<script setup>
jsegarra commented 2024-02-08 07:17:57 +00:00
Member
Copy Link

Desde el listado de termógrafos, si le das a editar te muestra el formulario pero:

  1. Te lanza esta llamada al api http://localhost:9000/api/TravelThermographs/[object%20Object]?filter=%7B%22include%22%3A%7B%22relation%22%3A%22dms%22%7D%7D
    . Como puedes ver tienes [object%20Object]
  2. La URL del edit queda algo así http://localhost:9000/#/travel/2/thermographs/[object%20Object]/edit
Desde el listado de termógrafos, si le das a editar te muestra el formulario pero: 1. Te lanza esta llamada al api http://localhost:9000/api/TravelThermographs/[object%20Object]?filter=%7B%22include%22%3A%7B%22relation%22%3A%22dms%22%7D%7D . Como puedes ver tienes [object%20Object] 2. La URL del edit queda algo así http://localhost:9000/#/travel/2/thermographs/[object%20Object]/edit
wbuezas commented 2024-02-08 11:23:51 +00:00
Contributor
Copy Link

Corregido.

Commit: 770f17a362

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/770f17a3624ff4f8a08f677f9e90e02d2da3ba6c
jsegarra marked this conversation as resolved
src/pages/Travel/Card/TravelThermographsForm.vue
@ -0,0 +1,325 @@
<script setup>
import { useI18n } from 'vue-i18n';
jsegarra commented 2024-02-08 07:19:03 +00:00
Member
Copy Link

Si desde el listado de termógrafos, le das a eliminar te dice:

  1. Que ha habido un error
  2. Que se ha eliminado
  3. No elimina el registro de la tabla.
Si desde el listado de termógrafos, le das a eliminar te dice: 1. Que ha habido un error 2. Que se ha eliminado 3. No elimina el registro de la tabla.
wbuezas commented 2024-02-08 11:19:16 +00:00
Contributor
Copy Link

@jsegarra en salix me da el mismo comportamiento que en lilium, por ejemplo si vas a los thermographs del first travel

Aparecen 4 registros donde dos te deja borrar exitosamente y dos arroja un error el backend.

Adjunto captura tomada de salix:

@jsegarra en salix me da el mismo comportamiento que en lilium, por ejemplo si vas a los thermographs del [first travel](http://localhost:9000/travel/3/summary#/travel/1/thermographs/index?limit=10) Aparecen 4 registros donde dos te deja borrar exitosamente y dos arroja un error el backend. Adjunto captura tomada de `salix`:
wbuezas commented 2024-02-08 11:27:42 +00:00
Contributor
Copy Link

Se fixea el problema de las notificaciones.

Commit: f24b9f5a75

Se fixea el problema de las notificaciones. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/f24b9f5a75e55eb17903482da337d440189bcdb0
jsegarra commented 2024-02-09 05:19:38 +00:00
Member
Copy Link
  1. Si subes un fichero que es de tipo zip por ejemplo sólo te muestra el triangulo de advertencia en un toast rojo.
  2. Al crear te redirige a index pero la consola muestra errores. No sé muy bien si es por los errores o por el error que ocurre en mas partes de la aplicación.
  3. Darle a eliminar termógrafo, da error porque no se pasa el Id
1. Si subes un fichero que es de tipo zip por ejemplo sólo te muestra el triangulo de advertencia en un toast rojo. 2. Al crear te redirige a index pero la consola muestra errores. No sé muy bien si es por los errores o por el error que ocurre en mas partes de la aplicación. 3. Darle a eliminar termógrafo, da error porque no se pasa el Id
jsegarra marked this conversation as resolved
src/pages/Travel/Card/TravelThermographsForm.vue
@ -0,0 +234,4 @@
<QCard class="q-pa-lg">
<VnRow class="row q-gutter-md q-mb-md">
<div class="col">
<VnSelectFilter
jsegarra commented 2024-02-08 06:57:35 +00:00
Member
Copy Link

No hay opción de crear termógrafo
Y por tanto no se puede evaluar el diálogo

No hay opción de crear termógrafo Y por tanto no se puede evaluar el diálogo
wbuezas commented 2024-02-08 11:07:53 +00:00
Contributor
Copy Link

Probe y pude acceder con el boton (+) que esta abajo a la derecha, que se encuentra en la vista de TravelThermographs, no te aparecio a vos?

Probe y pude acceder con el boton `(+)` que esta abajo a la derecha, que se encuentra en la vista de `TravelThermographs`, no te aparecio a vos?
wbuezas commented 2024-02-09 02:34:50 +00:00
Contributor
Copy Link

@jsegarra ahora si, cree el form necesario para crear un thermograph y lo aplique al input correspondiente.

Ademas de esto que era el requerimiento principal, implemente algunas cosas como:

  • Un estado de error en los inputs VnInput y VnSelectFilter cuando se le agrega la prop :required="true"
  • Algunas traducciones al objeto global para reutilizar
  • Implementacion del icon (i) con informacion sobre los tipos de files aceptados por el input QFile en el componente TravelThermographsForm

Commit: c67f4cf858

@jsegarra ahora si, cree el form necesario para crear un `thermograph` y lo aplique al input correspondiente. Ademas de esto que era el requerimiento principal, implemente algunas cosas como: - Un estado de error en los inputs `VnInput` y `VnSelectFilter` cuando se le agrega la prop `:required="true"` - Algunas traducciones al objeto global para reutilizar - Implementacion del icon `(i)` con informacion sobre los tipos de files aceptados por el input `QFile` en el componente `TravelThermographsForm` Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/c67f4cf858897cd2684946f44bd02c6eae808c68
jsegarra commented 2024-02-09 05:10:33 +00:00
Member
Copy Link

Se ve mal. Esto creo que ya nos pasó en otra PR antigua, creo que PR-3

Se ve mal. Esto creo que ya nos pasó en otra PR antigua, creo que PR-3
jsegarra commented 2024-02-09 05:27:09 +00:00
Member
Copy Link

Falta traducir el título del modal

Falta traducir el título del modal
wbuezas commented 2024-02-09 11:53:44 +00:00
Contributor
Copy Link

Listo! Ambos comentarios corregidos.

Además de esto me atreví a implementar la posibilidad de agregar un Tooltip al ícono de acción del input VnSelectDialog si se desea

Commit: 1b31657338

Listo! Ambos comentarios corregidos. Además de esto me atreví a implementar la posibilidad de agregar un `Tooltip` al ícono de acción del input `VnSelectDialog` si se desea Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/1b316573385f08d51a2b8279ace91d71732fb6d6
jsegarra marked this conversation as resolved
src/pages/Travel/Card/TravelThermographsForm.vue Outdated
@ -0,0 +297,4 @@
/>
</div>
</VnRow>
<VnRow class="row q-gutter-md q-mb-md">
jsegarra commented 2024-02-08 06:58:35 +00:00
Member
Copy Link

El icono del clip debería funcionar como cuando haces clic en el input.
Esto está subsanado en varios sitios.

El icono del clip debería funcionar como cuando haces clic en el input. Esto está subsanado en varios sitios.
wbuezas commented 2024-02-08 11:02:10 +00:00
Contributor
Copy Link

Corregido.

Commit: c8153b6414

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/c8153b641463dced58dc2e31a99744823d6c259c
jsegarra marked this conversation as resolved
src/pages/Travel/TravelList.vue
@ -1,5 +1,5 @@
<script setup>
jsegarra commented 2024-02-08 06:43:01 +00:00
Member
Copy Link

Cuando pulsas en un detalle de un travel, se muestra el dialogo.
Si pulsas sobre el primer icono:

  1. No tiene tooltip
  2. Redirige a salix y no a lilium. @alexm es correcto?
  3. Redirige mal, pues le falta el summary después.
Cuando pulsas en un detalle de un travel, se muestra el dialogo. Si pulsas sobre el primer icono: 1. No tiene tooltip 2. Redirige a salix y no a lilium. @alexm es correcto? 3. Redirige mal, pues le falta el summary después.
wbuezas commented 2024-02-08 10:59:27 +00:00
Contributor
Copy Link

Corregido.

Commit: 4dc31bfdce

Corregido. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/4dc31bfdcec15f752a40ef88df4fd70282230a29
jsegarra commented 2024-02-08 11:33:43 +00:00
Member
Copy Link

1 y 3 OK

1 y 3 OK
jsegarra marked this conversation as resolved
wbuezas added 1 commit 2024-02-08 10:59:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4dc31bfdce Travel summary corrections
wbuezas added 1 commit 2024-02-08 11:01:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
c8153b6414 Add action to qfile icon
wbuezas added 1 commit 2024-02-08 11:21:58 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
770f17a362 Fix edit thermograph id problem
wbuezas added 1 commit 2024-02-08 11:26:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f24b9f5a75 Adjust thermograph notify
wbuezas added 1 commit 2024-02-09 02:30:20 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
c67f4cf858 Thermograph create form and general improvements
alexm approved these changes 2024-02-09 07:50:43 +00:00
Dismissed
wbuezas added 1 commit 2024-02-09 11:51:52 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
1b31657338 Add tooltip to VnSelegDialog Action icon and small corrections
wbuezas dismissed alexm’s review 2024-02-09 11:51:53 +00:00
Reason:

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

wbuezas added 1 commit 2024-02-09 11:59:58 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
399f47b17c merge with salix-front dev branch and resolve conflicts
jsegarra approved these changes 2024-02-09 13:02:15 +00:00
alexm approved these changes 2024-02-09 13:05:24 +00:00
jsegarra merged commit c250f2c154 into dev 2024-02-09 13:11:23 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
alexm
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#185
No description provided.
Delete Branch ":feature/FaltantesModuloTravel"

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?