verdnatura/hedera-web
verdnatura
/
hedera-web
0
1
Fork 3
Code Issues Pull Requests 3 Releases Wiki Activity

refactor: use Salix to create or edit an Order #132

Merged
jsegarra merged 16 commits from ldragan/hedera-web:taro/sc into beta 2025-04-16 22:01:49 +00:00
Conversation 0 Commits 16 Files Changed 7 +17 -11
1 changed files with 17 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c5fbb9205a - Show all commits

28
src/pages/Ecomerce/CheckoutView.vue
View File

@ -313,17 +313,23 @@ const configureOrder = (orderId) => api.post(
},
);
const createOrder = (userId) => api.post(
'Orders',
{
sourceApp: 'WEB',
landed: new Date(orderForm.value.date),
clientFk: userId,
companyFk: 442, // SELECT defaultCompanyFk FROM orderConfig; (wtf)
addressFk: orderForm.value.address,
agencyModeFk: orderForm.value.agency,
},
);
const createOrder = async (userId) => {
const orderConfig = await api.get('OrderConfigs');
const companyFk = orderConfig.data[0]?.defaultCompanyFk;
jsegarra commented 2025-04-12 07:34:32 +00:00
Review
Copy Link

async?

async?
ldragan commented 2025-04-14 01:51:24 +00:00
Review
Copy Link

No es necesario porque no usamos await dentro de configureOreder.

Preferís que lo agregue, igualmente?

No es necesario porque no usamos `await` dentro de `configureOreder`. Preferís que lo agregue, igualmente?
jsegarra commented 2025-04-14 07:57:23 +00:00
Review
Copy Link

Esta bien, no lo había visto. De la otra manera seria redundante

Esta bien, no lo había visto. De la otra manera seria redundante
❤️ 1
return api.post(
'Orders',
{
sourceApp: 'WEB',
landed: new Date(orderForm.value.date),
clientFk: userId,
companyFk,
addressFk: orderForm.value.address,
agencyModeFk: orderForm.value.agency,
},
)
};
const submit = async () => {
loading.value = true;