feat: refs #8463 cardDescriptorBeta #1291

Closed
alexm wants to merge 10 commits from 8463-CardDescriptor_useCard into dev
Member
No description provided.
alexm added 1 commit 2025-01-27 13:21:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a6a2723734
feat: refs #8463 cardDescriptorBeta
alexm reviewed 2025-01-27 13:22:13 +00:00
@ -56,1 +71,3 @@
});
watch(
() => arrayData?.value?.isLoading,
(loading) => !loading && emit('onFetch', arrayData.value.store.data),
Author
Member

Con este watcher hacemos que cuando el card cambie de estado, emita los datos.

Con este watcher hacemos que cuando el card cambie de estado, emita los datos.
alexm marked this conversation as resolved
alexm reviewed 2025-01-27 13:24:04 +00:00
@ -0,0 +73,4 @@
<template>
{{ id }}
<component
Author
Member

Usamos el card, pero pasandole visual a false

Usamos el card, pero pasandole visual a `false`
alexm reviewed 2025-01-27 13:24:38 +00:00
@ -4,4 +31,4 @@
</script>
<template>
<VnCardBeta
Author
Member

Ahora el card hace la peticion mas reusable para los modulos

Ahora el card hace la peticion mas reusable para los modulos
alexm marked this conversation as resolved
alexm reviewed 2025-01-27 13:25:42 +00:00
@ -16,0 +16,4 @@
v-if="$props.id"
:id="$props.id"
:summary="OrderSummary"
data-key="orderDescriptor"
Author
Member

hay que cambiar el dataKey cuando es popup. sino si estas en ticket y abres un ticketDescriptor, te usara el mismo arrayData para los dos

hay que cambiar el dataKey cuando es popup. sino si estas en ticket y abres un ticketDescriptor, te usara el mismo arrayData para los dos
Member

Si si, esto es vital

Si si, esto es vital
alexm reviewed 2025-01-27 13:26:03 +00:00
@ -107,3 +108,3 @@
<span class="link">
{{ dashIfEmpty(entity?.address?.nickname) }}
<CustomerDescriptorProxy :id="entity?.clientFk" />
<OrderDescriptorProxy :id="1" />
Author
Member

Lo he puesto para probar el funcionamiento en modo popup
Habria que quitarlo

Lo he puesto para probar el funcionamiento en modo popup Habria que quitarlo
alexm marked this conversation as resolved
Author
Member

Habria que dejar en el card tambien la parte del orderTotal.
Ahora mismo se usa en 2-3 sitios.

❗Habria que dejar en el card tambien la parte del orderTotal. Ahora mismo se usa en 2-3 sitios.
alexm reviewed 2025-01-27 13:30:08 +00:00
@ -66,3 +81,1 @@
<div :class="[useCardSize(), $attrs.class]">
<RouterView :key="route.path" />
</div>
<span v-if="visual">
Author
Member

No me acaba este enfoque, pero he visto y probado que con Vue no se puede acceder a funciones de un componente sin montarlo.
Seria esta opcion o dividir la logica del template...

No me acaba este enfoque, pero he visto y probado que con Vue no se puede acceder a funciones de un componente sin montarlo. Seria esta opcion o dividir la logica del template...
alexm added 1 commit 2025-03-11 11:55:52 +00:00
alexm added 2 commits 2025-03-13 08:01:19 +00:00
alexm added 1 commit 2025-03-13 08:05:26 +00:00
alexm added 1 commit 2025-03-13 09:30:43 +00:00
alexm reviewed 2025-03-13 09:33:45 +00:00
@ -16,10 +19,13 @@ const props = defineProps({
searchDataKey: { type: String, default: undefined },
searchbarProps: { type: Object, default: undefined },
redirectOnError: { type: Boolean, default: false },
visual: { type: Boolean, default: true },
Author
Member

Se añade una prop para hacer que el template sea visible o no (v-if)

Se añade una prop para hacer que el template sea visible o no (v-if)
Member

Sugerencia, la variable porque no se llamaba visible?

Sugerencia, la variable porque no se llamaba visible?
alexm reviewed 2025-03-13 09:34:44 +00:00
@ -53,3 +59,2 @@
}
const id = to.params.id;
if (id !== from.params.id) await fetch(id, true);
if (entityId.value !== to.params.id) await fetch(to.params.id, true);
Author
Member

He visto que asi funciona mejor. Si no hacia 1 1. Pq el entityId no sehabia actualizado antes cambiar de seccion

He visto que asi funciona mejor. Si no hacia 1 1. Pq el entityId no sehabia actualizado antes cambiar de seccion
alexm reviewed 2025-03-13 09:35:08 +00:00
@ -1,13 +1,9 @@
<script setup>
Author
Member

Al cardDescriptor le quitamos la funcionalidad del template

Al cardDescriptor le quitamos la funcionalidad del template
alexm reviewed 2025-03-13 09:35:33 +00:00
@ -0,0 +1,38 @@
<script setup>
Author
Member

Este es el nuevo Descriptor al que se le pasa el componente del Card para que lo use

Este es el nuevo Descriptor al que se le pasa el componente del Card para que lo use
alexm reviewed 2025-03-13 09:36:13 +00:00
@ -0,0 +1,296 @@
<script setup>
Author
Member

El template base para los 2 componentes

El template base para los 2 componentes
jsegarra reviewed 2025-03-13 09:40:49 +00:00
@ -71,3 +76,1 @@
<div :class="[useCardSize(), $attrs.class]">
<RouterView :key="$route.path" />
</div>
<span v-if="visual">
Member

la etiqueta span se suele usar para texto, en este caso usaria div

la etiqueta span se suele usar para texto, en este caso usaria div
alexm marked this conversation as resolved
jsegarra reviewed 2025-03-13 09:47:01 +00:00
@ -0,0 +158,4 @@
<QList dense>
<QItemLabel header class="ellipsis text-h5" :lines="1">
<div class="title">
<span v-if="$props.title" :title="getValueFromPath(title)">
Member

el parámetro de la función entiendo que viene de $props.title
Entonces porque viewSummary si que los pasas?

Ya que lo movemos podemos hacer cambios que antes no veiamos

el parámetro de la función entiendo que viene de $props.title Entonces porque viewSummary si que los pasas? Ya que lo movemos podemos hacer cambios que antes no veiamos
alexm marked this conversation as resolved
jsegarra reviewed 2025-03-13 10:51:37 +00:00
@ -0,0 +286,4 @@
}
}
</style>
<i18n>
Member

esto debería estar aquí?

esto debería estar aquí?
alexm marked this conversation as resolved
alexm added 1 commit 2025-03-14 07:59:28 +00:00
alexm added 1 commit 2025-03-14 08:13:50 +00:00
alexm added 1 commit 2025-03-14 08:15:55 +00:00
alexm added 1 commit 2025-03-14 08:37:04 +00:00
alexm changed title from WIP: feat: refs #8463 cardDescriptorBeta to feat: refs #8463 cardDescriptorBeta 2025-03-14 08:53:38 +00:00
alexm closed this pull request 2025-03-14 08:53:45 +00:00
Some checks reported warnings
gitea/salix-front/pipeline/pr-dev This commit is unstable
Required
Details

Pull request closed

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#1291
No description provided.