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

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
Conversation 0 Commits 3 Files Changed 2 +3 -3
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/ui/VnNotes.vue
View File

@ -78,8 +78,8 @@ async function insert() {
<TransitionGroup name="list" tag="div" class="column items-center full-width"> <TransitionGroup name="list" tag="div" class="column items-center full-width">
<QCard <QCard
class="q-pa-xs q-mb-sm full-width" class="q-pa-xs q-mb-sm full-width"
v-for="note in rows" v-for="(note, index) in rows"
:key="note.id" :key="note.id ?? index"
jorgep commented 2024-04-22 07:01:40 +00:00
Outdated
Review
Copy Link

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.
alexm commented 2024-04-22 07:36:37 +00:00
Outdated
Review
Copy Link

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
jorgep commented 2024-04-22 07:40:21 +00:00
Outdated
Review
Copy Link

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.
> >
<QCardSection horizontal> <QCardSection horizontal>
<VnAvatar <VnAvatar

2
src/pages/Claim/Card/ClaimNotes.vue
View File

@ -16,7 +16,7 @@ const claimId = computed(() => $props.id || route.params.id);
const claimFilter = { const claimFilter = {
where: { claimFk: claimId.value }, where: { claimFk: claimId.value },
fields: ['created', 'workerFk', 'text'], fields: ['id', 'created', 'workerFk', 'text'],
include: { include: {
relation: 'worker', relation: 'worker',
scope: { scope: {