Add purchase callback #34
Conversation
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/[email protected] |
| } | ||
|
|
||
| const { data, errors } = | ||
| await client.mutate<CheckPurchaseOrderStatusMutation>({ |
There was a problem hiding this comment.
Quizas podríamos usar el useCheckPurchaseOrderStatusMutation(), y llamar la función dentro de un useEffect.
Como porque si este componente se re-renderiza onda... X veces, se manda la mutación X veces también.
(O llamar client.mutate() dentro de un useEffect también funcionaría, pero es mas comun usar el useCheckPurchaseOrderStatusMutation)
There was a problem hiding this comment.
lo habia intentado asi al principio, pero entraba en conflicto con el SSR. Si lo transformo a un client component podría usar useEffect.
| <CardTitle>Oops!</CardTitle> | ||
| <CardContent> | ||
| <ol> | ||
| {errors.map((error, index) => ( |
There was a problem hiding this comment.
nit: derepente esto va a exponer como errores onda internos o no? 🤔
There was a problem hiding this comment.
sip, pero quiero dejarlo como placeholder y mejorarlo despues: #35
Adds a simple purchase callback, to manage the signal back from commerce (Stripe/MercadoPago) and trigger CheckPurchaseOrderStatus mutation