WIP: feat: #8406 upgraded CrudModel #1627

Draft
provira wants to merge 5 commits from 8406-crudModelUpdate into dev
Member
No description provided.
provira added 1 commit 2025-03-24 10:06:58 +00:00
provira added 1 commit 2025-03-24 10:07:04 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
5b42e97d83
Merge branch 'dev' into 8406-crudModelUpdate
provira requested review from alexm 2025-03-24 11:03:50 +00:00
alexm requested changes 2025-03-27 07:06:47 +00:00
@ -46,0 +50,4 @@
},
insertOnLoad: {
type: Boolean,
default: true,
Member

default false, y donde se quiera, poner true

default false, y donde se quiera, poner true
Member

@jgallego que opinas?

@jgallego que opinas?
Owner

la mayoria de los campos no se usan para crear nuevos registros, por tanto, por defecto false creo que es mejor, además el hecho de quererlo en el formulario de creación para mi tiene que ser explicito. Por tanto, default: false

la mayoria de los campos no se usan para crear nuevos registros, por tanto, por defecto false creo que es mejor, además el hecho de quererlo en el formulario de creación para mi tiene que ser explicito. Por tanto, default: false
provira marked this conversation as resolved
@ -138,1 +155,3 @@
watchChanges.value = watch(formData, () => (hasChanges.value = true), { deep: true });
watchChanges.value = watch(formData, (nVal) => {
hasChanges.value = false;
for(let index = 0; index < nVal.length; index++) {
Member

Hay unas funciones para sacar los cambios
import { getDifferences, getUpdatedValues } from 'src/filters';

Las usaria para esto tambien

Hay unas funciones para sacar los cambios `import { getDifferences, getUpdatedValues } from 'src/filters';` Las usaria para esto tambien
Member

Me suena que esto lo miramos juntos... Prueba a ver si se puede simplificar con las funciones que dice Alex.

Me suena que esto lo miramos juntos... Prueba a ver si se puede simplificar con las funciones que dice Alex.
provira marked this conversation as resolved
@ -210,2 +239,2 @@
formData.value.push(Object.assign({ $index }, pushData));
hasChanges.value = true;
async function insert(pushData = { ...$props.dataRequired, ...$props.dataDefault }) {
const lastRow = formData.value.at(-1);
Member

Para ignorar las líneas vacías, usaría un filter
Para quitar las líneas vacías, usaría un map

Para ignorar las líneas vacías, usaría un filter Para quitar las líneas vacías, usaría un map
provira marked this conversation as resolved
@ -221,3 +276,3 @@
const pk = $props.primaryKey;
let ids = data.map((d) => d[pk]).filter(Boolean);
let preRemove = data.map((d) => (d[pk] ? null : d.$index)).filter(Boolean);
let preRemove = data.map((d) => (d[pk] ? null : d.$index)).filter(index => index !== null && index !== undefined);
Member

Creo que hace lo mismo

Creo que hace lo mismo
provira marked this conversation as resolved
@ -250,3 +305,3 @@
});
} else {
reset();
await fetch(formData.value);
Member

si no hay filas, fetch? Se tiene original data

si no hay filas, fetch? Se tiene original data
provira marked this conversation as resolved
alexm requested review from jorgep 2025-03-27 07:06:52 +00:00
Member
  • si hay una nueva fila y está vacía, no se tiene que detectar como cambio.
  • no poder insertar más de una fila vacía. Valorar implementarlo a nivel global de qTable o solo de vnTable. Las secciones sin vnTable seguiran con este error.
  • Establecer por defecto en las tablas que tengan CRUDModel , el mini btn "+" en la parte posterior izquierda, que inserte una fila nueva vacía.
  • Al ir tabulando entre campos, si estás al final de la fila y vuelves a tabular que te añada una fila vacía.
  • Al entrar en la sección que añada una fila nueva(vacía) directamente.
- [x] si hay una nueva fila y está vacía, no se tiene que detectar como cambio. - [x] no poder insertar más de una fila vacía. Valorar implementarlo a nivel global de qTable o solo de vnTable. Las secciones sin vnTable seguiran con este error. - [ ] Establecer por defecto en las tablas que tengan CRUDModel , el mini btn "+" en la parte posterior izquierda, que inserte una fila nueva vacía. - [x] Al ir tabulando entre campos, si estás al final de la fila y vuelves a tabular que te añada una fila vacía. - [x] Al entrar en la sección que añada una fila nueva(vacía) directamente.
jorgep requested changes 2025-03-28 12:06:27 +00:00
jorgep left a comment
Member

Faltan puntos por hacer. Te he puesto los checkbox en un comentario de la pr.

Faltan puntos por hacer. Te he puesto los checkbox en un comentario de la pr.
provira added 1 commit 2025-04-03 06:39:11 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
87c1c16d26
feat: refs #8406 added new insertion with tabulation
provira added 1 commit 2025-04-03 06:52:27 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
c075f39576
Merge branch 'dev' into 8406-crudModelUpdate
provira added 1 commit 2025-04-03 07:57:55 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
5703798781
fix: refs #8406 fixed front test
jsegarra changed title from feat: #8406 upgraded CrudModel to WIP: feat: #8406 upgraded CrudModel 2025-04-04 07:12:03 +00:00
Some checks reported warnings
gitea/salix-front/pipeline/pr-dev This commit is unstable
Required
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
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#1627
No description provided.