Add order purchase callback #37
Conversation
Signed-off-by: Hector Vergara <[email protected]>
Deploying tickets with
|
| Latest commit: |
a2c8f72
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7c9d62c7.tickets.pages.dev |
| Branch Preview URL: | https://hvergara-purchase-callback.tickets.pages.dev |
|
|
||
| useEffect(() => { | ||
| if (!triggered.current) { | ||
| void checkPurchaseOrder(); |
There was a problem hiding this comment.
Podemos llamar al checkPurchaseOrderMutation directo y nos ahorramos un useCallback.
Porque el checkPurchaseOrderMutation no cambia si se renderiza el componente.
|
|
||
| useEffect(() => { | ||
| if (!triggered.current) { | ||
| void checkPurchaseOrder(); |
There was a problem hiding this comment.
Tangencial, pero re pocas veces veo void 🤔 no se si derepente me estoy perdiendo algo... Pero si no estamos retornando checkPurchaseOrder es necesario? O es mas estilistico?
| export const PurchaseCallback = ({ | ||
| purchaseOrderId, | ||
| }: PurchaseCallbackProps) => { | ||
| const triggered = useRef(false); |
There was a problem hiding this comment.
No tenemos un estandar definido, pero prefiero usar refs para dom nodes, o referencias que necesitemos que no se regeneren. (Onda setInterval por ejemplo...)
Podemos conversarlo, pero soy mas partidario de errar hacia el useState por convención/default
| const purchaseOrderId = params.get("purchaseOrderId"); | ||
|
|
||
| if (!purchaseOrderId) { | ||
| return toast("Error fetching purchase order"); |
There was a problem hiding this comment.
toast retorna undefined, asi que tecnicamente esto esta bien. Pero como toast no es un componente, creo que seria mejor o un poco mas claro, que retornaramos null aca y que dejaramos el toast en un useEffect
useEffect(() => {
if(!purchaseOrderId) {
toast("Error fetching purchase order");
}
}, [purchaseOrderId])
Improves purchase callback logic. Note: design & localization will be improved in future PRs, setting up minimal UI for now.