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

CambiosSolicitadosSuppliers #214

Merged
jgallego merged 21 commits from :CambiosSolicitadosSuppliers into dev 2024-03-13 14:38:00 +00:00
Conversation 8 Commits 21 Files Changed 34 +4 -4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 052123ad0e - Show all commits

8
src/pages/Supplier/Card/SupplierDescriptor.vue
View File

@ -73,7 +73,7 @@ const setData = (entity) => {
data.value = useCardDescription(entity.ref, entity.id);
};
const currentSupplier = computed(() => state.get('supplier'));
const supplier = computed(() => state.get('supplier'));
jsegarra marked this conversation as resolved Outdated
jgallego commented 2024-03-04 11:38:09 +00:00
Outdated
Review
Copy Link

si cogemos el patron de usar entity igual que parece que hacia salix para guadar la entidad en la que nos encontramos mejor, @jsegarra confirma tu.

si cogemos el patron de usar entity igual que parece que hacia salix para guadar la entidad en la que nos encontramos mejor, @jsegarra confirma tu.
👀 2
jsegarra commented 2024-03-05 12:52:54 +00:00
Outdated
Review
Copy Link

@wbuezas A lo que se refiere Javi es al cambio del nombre de la variable? Hay algún motivo explicito?

La verdad es que si ya estas en el descriptor sabes que es el currentSupplier. Quizás si fuese un formulario en el que manejas diferentes estados puedo entenderlo.

Por lo que he visto, al declarar como entity puede dar problemas de reactividad con los iconos

Aunque vemos que no se está usando este patrón en ClaimDescriptor, por ello, podemos cambiar currentsupplier por supplier?

@wbuezas A lo que se refiere Javi es al cambio del nombre de la variable? Hay algún motivo explicito? La verdad es que si ya estas en el descriptor sabes que es el currentSupplier. Quizás si fuese un formulario en el que manejas diferentes estados puedo entenderlo. Por lo que he visto, al declarar como entity puede dar problemas de reactividad con los iconos Aunque vemos que no se está usando este patrón en ClaimDescriptor, por ello, podemos cambiar currentsupplier por supplier?
wbuezas commented 2024-03-05 13:07:30 +00:00
Outdated
Review
Copy Link

Claro, tomé la decisión de cambiar el nombre de la variable porque en el html ya utilizamos la variable entity que se obtiene del v-slot y en este caso necesitaba obtener la información del store de manera que se podian repetir/pisar las variables con el mismo nombre

De todas maneras cambié el nombre de la variable currentSupplier por el nombre supplier

Commit: 052123ad0e

Claro, tomé la decisión de cambiar el nombre de la variable porque en el html ya utilizamos la variable `entity` que se obtiene del `v-slot` y en este caso necesitaba obtener la información del store de manera que se podian repetir/pisar las variables con el mismo nombre De todas maneras cambié el nombre de la variable `currentSupplier` por el nombre `supplier` Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/052123ad0e6259d6e08b53bbc82352a1eae78e1b
🎉 1
const getEntryQueryParams = (supplier) => {
if (!supplier) return null;
@ -138,9 +138,9 @@ const getEntryQueryParams = (supplier) => {
<VnLv :label="t('supplier.summary.account')" :value="entity.account" />
</template>
<template #icons>
<QCardActions v-if="currentSupplier" class="q-gutter-x-md">
<QCardActions v-if="supplier" class="q-gutter-x-md">
<QIcon
v-if="!currentSupplier.isActive"
v-if="!supplier.isActive"
name="vn:disabled"
color="primary"
size="xs"
@ -148,7 +148,7 @@ const getEntryQueryParams = (supplier) => {
<QTooltip>{{ t('Inactive supplier') }}</QTooltip>
</QIcon>
<QIcon
v-if="!currentSupplier.isSerious"
v-if="!supplier.isSerious"
name="vn:supplierfalse"
color="primary"
size="xs"