fix(VnNotes): refs #6336 fix v-for key #310

Merged
alexm merged 3 commits from 6336_fix_vn_notes_key into dev 2024-04-22 12:09:19 +00:00
Member
No description provided.
alexm added 1 commit 2024-04-22 06:43:04 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
99af402281
fix(VnNotes): refs #6336 fix v-for key
alexm requested review from jorgep 2024-04-22 06:43:14 +00:00
jorgep reviewed 2024-04-22 07:01:40 +00:00
@ -81,2 +81,2 @@
v-for="note in rows"
:key="note.id"
v-for="(note, index) in rows"
:key="note.id ?? index"
Member

Las notas siempre tienen id no? El error parece que venía de no añadir el id en fields. En caso de poder ser null Lo veo ok. Pero creo que es mejor que sea siempre el id de la nota o el index, no mezclado.

Las notas siempre tienen id no? El error parece que venía de no añadir el id en fields. En caso de poder ser null Lo veo ok. Pero creo que es mejor que sea siempre el id de la nota o el index, no mezclado.
Author
Member

La cosa es que si alguien no pone id, va a dar warnings i hace que la seccion vaya petada...
Pero bueno confiamos en que la gente ponga el id jeje

La cosa es que si alguien no pone id, va a dar `warnings` i hace que la seccion vaya petada... Pero bueno confiamos en que la gente ponga el `id` jeje
Member

Pues pon siempre el index. Index siempre va a ver. Te lo decía porque no de lugar a confusión. Por que podría darse el caso de que el primer registro tenga id "2" y el segundo registro no tenga id y tenga index "2". Entonces es confuso y puede que de error.

Pues pon siempre el index. Index siempre va a ver. Te lo decía porque no de lugar a confusión. Por que podría darse el caso de que el primer registro tenga id "2" y el segundo registro no tenga id y tenga index "2". Entonces es confuso y puede que de error.
alexm added 1 commit 2024-04-22 07:38:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
918ce0dc86
fix(VnNotes): refs #6336 remove index in key
alexm added 1 commit 2024-04-22 07:46:47 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4a5285358c
fix(VnNotes): refs #6336 restore index in key
jorgep approved these changes 2024-04-22 08:08:15 +00:00
alexm merged commit 9d381a5c82 into dev 2024-04-22 12:09:19 +00:00
alexm deleted branch 6336_fix_vn_notes_key 2024-04-22 12:09:19 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#310
No description provided.