fix: filtrar producto sin stock del banner y banco placeholder de PSE
Encontrado probando en vivo con navegador real (Docker + Playwright contra shop.cruzcloud.net): - app/page.tsx: un producto featured pero sin stock (available: false) salía primero en el hero banner con botón "Comprar ahora" hacia una página donde en realidad no se puede comprar. - pse-banks/route.ts: el sandbox real de ePayco (bank.getBanks()) incluye un primer elemento placeholder (bankCode "0", "A continuación seleccione su banco...") mezclado con los bancos reales, mostrado como opción seleccionable. Se filtra explícitamente. Co-Authored-By: Claude Sonnet 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01THZFwPzrwv6VAbuwmaf5vR
This commit is contained in:
1 parent
44e343eca7
commit
ff59618a2a
2 files changed
+14
-4
No files matched your search
@@ -12,6 +12,12 @@ import { createEpaycoClient, type EpaycoEnv } from "../../../../lib/epayco";
|
||||
// contra el sandbox real en esta sesion -- se prueban los nombres de
|
||||
// campo mas probables (bankCode/bankName y variantes) y se degrada a
|
||||
// lista vacia si no matchea nada, en vez de romper el checkout entero.
|
||||
//
|
||||
// Verificado en vivo despues (navegador real, Playwright): el sandbox
|
||||
// real de ePayco incluye un primer elemento placeholder
|
||||
// (bankCode "0", "A continuación seleccione su banco...") mezclado con
|
||||
// los bancos reales -- ningun banco PSE real tiene codigo "0", asi que
|
||||
// se filtra explicitamente en vez de mostrarlo como opcion seleccionable.
|
||||
const CACHE_TTL_MS = 6 * 60 * 60 * 1000;
|
||||
let cache: { banks: Array<{ pseCode: string; description: string }>; fetchedAt: number } | null = null;
|
||||
|
||||
@@ -42,7 +48,7 @@ export async function GET(req: MedusaRequest, res: MedusaResponse) {
|
||||
? { pseCode: String(pseCode), description: String(description ?? pseCode) }
|
||||
: null;
|
||||
})
|
||||
.filter((bank): bank is { pseCode: string; description: string } => bank !== null);
|
||||
.filter((bank): bank is { pseCode: string; description: string } => bank !== null && bank.pseCode !== "0");
|
||||
|
||||
cache = { banks, fetchedAt: Date.now() };
|
||||
res.json({ banks });
|
||||
|
||||
Reference in new issue
Block a user