8197-VnCardMain #1044
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1044
Loading…
Reference in New Issue
No description provided.
Delete Branch "8197-VnCardMain"
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?
❗ Solo se han migrado los modulos de Account y de Ticket
Los demas se haran en #8316 y #8322
Se ha hecho codigo retrocompatible para que no fallen mientras no esten migradas
@ -49,10 +49,16 @@ export const useArrayDataStore = defineStore('arrayDataStore', () => {
});
}
function resetPagination(key) {
He visto que se repetia varias veces el patron y lo he hecho una funcion
@ -17,0 +26,4 @@
hasContent.value = teleportRef.value.innerHTML.trim() !== '';
};
observer = new MutationObserver(checkContent);
El hook update o alguno de esos no sirve? Entiendo que no, pero por si acaso pregunto
@ -15,2 +15,4 @@
() => (stateStore.leftDrawer = useQuasar().screen.gt.xs ? $props.leftDrawer : false)
);
const targetId = 'left-panel';
Para que quieres una variable que solo se usa en 1 sitio?
Antes de poner lo del ref la usaba 2 veces, lo quito
@ -20,3 +38,3 @@
<QDrawer v-model="stateStore.leftDrawer" show-if-above :width="256">
<QScrollArea class="fit text-grey-8">
<LeftMenu />
<div :id="targetId" ref="teleportRef"></div>
template en lugar de div es posible?
Al ser para un teleport siempre lo he visto con div, lo pruebo
No puede ser template, deja de funcionar
@ -161,2 +141,2 @@
order="id DESC"
:disable-option="{ card: true }"
<VnCardMain
:section="dataKey"
Que esta prop tenga el valor por defecto de la prop datakey
Me gusta como piensas
@ -66,2 +46,2 @@
}"
order="id DESC"
<VnCardMain
:section="dataKey"
Lo mismo que el AccountAcls
@ -122,2 +106,2 @@
:filter="filter"
order="id DESC"
<VnCardMain
:section="dataKey"
Lo mismo que AccountAcls
@ -102,2 +88,2 @@
order="id ASC"
:disable-option="{ card: true }"
<VnCardMain
:section="dataKey"
Lo mismo que AccountAcls
@ -44,3 +44,3 @@
module="Role"
@on-fetch="setData"
data-key="accountData"
data-key="Role"
RoleId
No hace falta poner roleId, dado que el dataKey del list es AccountRoleList
Siendo Role ya va guay
@ -31,3 +31,3 @@
:filter="filter"
@on-fetch="(data) => (role = data)"
data-key="RoleSummary"
data-key="Role"
RoleId
@ -1,4 +1,7 @@
import { RouterView } from 'vue-router';
import accountCard from './account/accountCard';
Consultaría a Juan si le parece bien separarlo en varios ficheros. Por mí bien
Entiendo que lo verán guay sino se hace un monstruo de archivo y de todas maneras como estaba ya estaba separado
.
@ -0,0 +58,4 @@
<Teleport to="#left-panel" v-if="stateStore.isHeaderMounted()">
<component :is="descriptor" />
<QSeparator />
<LeftMenu source="card" />
No me acaba de gustar poner leftMenu en 2 sitios. No se le puede pasar el source en el meta o algo así? Como veo que este problema ya existía antes okey... Pero yo crearía tarea o algo...
Con el refactor que he hecho ya no hará falta poner el source pq te buscara de abajo a arriba. Me lo apunto en la tarea
Los nombres VnSection y VnModule me suenan a lo mismo... En lugar de VnModule, VnAside... o LeftMenu y cambiar este a LeftMenuModules / LeftMenuItems. Pero creo que yo optaría por VnAside y que lo usen LeftMenu y RightMenu. Como minimo LeftMenu que parece que es lo que te hace falta.
@ -28,3 +26,4 @@
const roles = ref();
const validationsStore = useValidator();
const { models } = validationsStore;
const dataKey = 'AccountAcls';
Quitar y poner directamente en la prop, solo se usa 1 vez
Quitar
@ -8,3 +7,3 @@
const tableRef = ref();
const { t } = useI18n();
const stateStore = useStateStore();
const dataKey = 'AccountAliasList';
quitar, solo se usa 1 vez.
@ -13,3 +11,4 @@
const filter = {
include: { relation: 'role', scope: { fields: ['id', 'name'] } },
};
const dataKey = 'AccountList';
quitar, solo se usa 1 vez.
@ -16,2 +18,4 @@
});
const tableRef = ref();
const url = 'VnRoles';
const dataKey = 'AccountRoleList';
quitar, solo se usa 1 vez.
@ -10,3 +12,4 @@
const quasar = useQuasar();
onMounted(async () => {
stateStore.rightDrawer = true;
En vista móvil se quedará abierto también.
Diria que es lo correcto pq hasta que no eliges video, no muestra nada
@ -54,1 +56,4 @@
}
return {
state,
Se te ha colado al hacer pruebas o necesitas que se vea la store entera?
Si, lo quito
Los nombres han sido validados con Javi, VnSection para secciones y VnModule pq es el modulo entero
Quitar variable de datakey