feat: refs #8077 sumDefaulter #1239
No reviewers
jsegarra
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1239
Loading…
Reference in New Issue
No description provided.
Delete Branch "8077-sumDefaulterFrontFix"
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?
verdnatura/salix#3367
@ -116,3 +117,1 @@resetData(data);emit('onFetch', data);return data;const rows = $attrs['key-data'] ? data[$attrs['key-data']] : data;No se ha hecho de esta forma por que puede que data este vacío
const rows = data[$attrs['key-data']] ?? data;
Lo que no me acaba de convencer de esto es usar una clave dos veces.
Quizás si fuese en un componente, con 2 ocurrencias...en el limite, pero bien; sin embargo, se trata de CrudModel que es transversal para todos y por ello, te propongo estas líneas
const keyData = $attrs['key-data']; const rows = keyData ? data[keyData] : data;@ -316,2 +316,3 @@<QScrollArea class="fit"><VnTableFilter :data-key="$attrs['data-key']" :columns="columns" :redirect="redirect" /><VnTableFilter:data-key="$attrs['data-key']"Arriba es key-data y aqui data-key? Alguna razón por hacerlo diferente?
DataKey y KeyData no son la misma propiedad. Se podria buscar otro nombre pero en este caso no se me ocurre pq literalmente queremos la "clave" del "data" jaja
New commits pushed, approval review dismissed automatically according to repository settings