#7750 - first-beta-review #95

Merged
jsegarra merged 26 commits from wbuezas/hedera-web-mindshore:first-beta-review into beta 2024-12-09 23:18:07 +00:00
1 changed files with 20 additions and 0 deletions
Showing only changes of commit 221d0e7b8c - Show all commits

View File

@ -28,6 +28,7 @@ const agencies = ref([]);
const warehouses = ref([]);
const currentStep = ref('method');
const id = route.params.id;
const defaultValues = ref(null);
const orderForm = ref({
method: 'AGENCY',
date: formatDate(Date.vnNew(), 'YYYY/MM/DD'),
@ -179,6 +180,15 @@ const getAgencies = async () => {
}
);
agencies.value = results[1].data;
if (agencies.value && agencies.value.length && defaultValues.value) {
const found = agencies.value.find(
agency => agency.id === defaultValues.value.defaultAgencyFk
);
Review

Duda, si no se encuentra no debería mostrar mensaje de error al usuario?
Cambiar found por agency

Duda, si no se encuentra no debería mostrar mensaje de error al usuario? Cambiar found por agency
Review

No creo que se deba mostrar un error por esto, supongo que por algo el hedera viejo no lo hacía

No creo que se deba mostrar un error por esto, supongo que por algo el hedera viejo no lo hacía
if (found)
orderForm.value.agency = defaultValues.value.defaultAgencyFk;
}
} catch (error) {
console.error('Error getting agencies:', error);
}
@ -284,6 +294,13 @@ const submit = async () => {
}
};
const getDefaultValues = async () => {
return await jApi.query(
`SELECT deliveryMethod, agencyModeFk, addressFk, defaultAgencyFk
FROM myBasketDefaults`
);
};
onMounted(async () => {
today.value = Date.vnNew();
today.value.setHours(0, 0, 0, 0);
@ -305,6 +322,9 @@ onMounted(async () => {
orderForm.value.agency = order.agencyModeFk;
orderForm.value.address = order.addressFk;
}
} else {
const [_defaultValues] = await getDefaultValues();
if (_defaultValues) defaultValues.value = _defaultValues;
}
getAddresses();