6157-actionClaim #106
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#106
Loading…
Reference in New Issue
No description provided.
Delete Branch "6157-actionClaim"
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?
WIP: 6157-actionClaimto 6157-actionClaim@ -0,0 +27,4 @@
const destinationTypes = ref([]);
const destinations = ref([]);
const totalClaimed = ref(null);
const maxResponsibility = computed(() => 5);
Si es un numero li pots possar const maxResponsibility = 5;
Si que puedes,
Sin embargo, te diría que ese 5, quizás moverlo a un const DEFAULT_MAX_RESPONSABILITY = 5.
Ya me dices
@ -0,0 +45,4 @@
{
name: 'destination',
label: t('Destination'),
field: (row) => getDestination(row.claimDestinationFk),
Pq no pots possar row.claimDestinationFk ?
@ -0,0 +90,4 @@
},
]);
watchEffect(() => {
? si lo que fas es sumar no seria mes facil sumar en el onmouted y quan afegixes o borres una linea?
@ -0,0 +103,4 @@
return destinationTypes.value.find((type) => type.id == destinationId);
}
async function deleteSale(sale) {
Esta funcio no se crida mai
@ -0,0 +457,4 @@
.claim-action {
.q-table {
.q-select .q-field__input {
display: none !important;
Com?
@ -0,0 +482,4 @@
height: min-content;
}
}
.totalClaim {
Aço si ja esta en els descriptors, en els summarys te que haver alguna clase ja feta (si no fesla)
@ -0,0 +15,4 @@
});
it('should change destination', () => {
const rowData = [true, null, null, 'Bueno', null, null, null, null, null, null];
Crec q els ultims nulls no fan falta
@ -0,0 +20,4 @@
});
it('should regularize', () => {
cy.get('[title="Regularize"]').click();
Aci deuries comprovar que fa lo que esperes
@ -0,0 +22,4 @@
it('should regularize', () => {
cy.get('[title="Regularize"]').click();
});
});
Te falta per fer tests a un boto no?
I borrar per a netejar les fixtures
@ -269,3 +269,3 @@
<SkeletonTable v-if="!formData" />
<Teleport to="#st-actions" v-if="stateStore?.isSubToolbarShown()">
<QBtnGroup push class="q-gutter-x-sm">
<QBtnGroup push style="column-gap: 10px">
Este estilo, es el que corregimos que permitía separar los botones pero tenerlos pegados al margen?
Si es así, quizás, ese estilo, puede ser usado en un futuro o recurrente en otros componentes, por lo que la propiedad column-gap la movería a una clase
@ -31,3 +36,2 @@
const { t } = useI18n();
const entity = ref();
defineExpose({ fetch });
Esta tb borrala
@ -52,3 +62,1 @@
entity.value = null;
await fetch();
});
async function fetch() {}
Borrala si no se gasta ni fa res
@ -68,6 +68,7 @@ const arrayData = useArrayData(props.dataKey, {
limit: props.limit,
order: props.order,
userParams: props.userParams,
skip: 0,
Prova a no possarla no recorde pq la possarem
@ -25,1 +21,4 @@
pending: 'positive',
managed: 'warning',
Els introoooooooooooos, tens prettier?
@ -0,0 +108,4 @@
claimActionsForm.value.reload();
}
async function updateDestinations(claimDestinationFk) {
Lo revisamos!!
@ -0,0 +118,4 @@
}
}
async function updateDestination(claimDestinationFk, row) {
Y esto también!!
@ -0,0 +134,4 @@
if (claim.value.responsibility >= Math.ceil(maxResponsibility.value) / 2) {
dialogGreuge.value = true;
} else {
quasar.notify({
Lo que le pasas a notify podría ser una constante, no? Porque lo hacemos en la línea 148, 166, 195
@ -0,0 +258,4 @@
label-always
color="primary"
markers
:marker-labels="[
Si, los labels cambian, tienes que venir hasta el HTML?
Quizás, mejor definirlo en el apartado de
@ -0,0 +262,4 @@
{ value: 1, label: t('claim.summary.company') },
{ value: 5, label: t('claim.summary.person') },
]"
:min="1"
Podemos definir este valor como constante?
Tipo
DEFAULT_MIN_SLIDER =1
DEFAULT_MAX_SLIDER =5
@ -22,11 +22,6 @@ const $props = defineProps({
const entityId = computed(() => {
esto no se usa, no?
@ -69,6 +69,7 @@ function stateColor(code) {
}
const data = ref(useCardDescription());
const setData = (entity) => {
if (!entity) return;
Revisamos linea 65!
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings