Skip to content

Listado de productos con filtros - #4

Merged
NachoBasilio merged 2 commits into
devfrom
feat/catalogo-limpio
May 11, 2026
Merged

NachoBasilio merged 2 commits into
devfrom
feat/catalogo-limpio

Conversation

@NachoBasilio

Copy link
Copy Markdown
Contributor

Que hace este PR?

Incorpora el catalogo filtrable de Sacre con listado de productos, busqueda, categorias, ordenamiento y rango de precio.

Por que se hace?

Reemplaza el placeholder del catalogo por una experiencia navegable y separa la logica de filtrado/ordenamiento para que la page no acumule responsabilidad de negocio.

Como probarlo?

  • pnpm lint
  • pnpm typecheck
  • pnpm test --run

Alcance corregido

Este PR reemplaza al PR #3, que quedo contaminado con commits de landing y conflictos contra dev. Esta rama nace desde dev actualizado y contiene solo dos commits:

  • add: incorpora catalogo filtrable
  • test: cubre comportamiento del catalogo

Notas

  • La suite sigue mostrando el warning existente de Vitest: `--localstorage-file` sin path valido.
  • No se ejecuto build por regla operativa del proyecto.

@netlify

netlify Bot commented May 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for musical-cucurucho-b52132 ready!

Name Link
🔨 Latest commit f66c68e
🔍 Latest deploy log https://app.netlify.com/projects/musical-cucurucho-b52132/deploys/6a02668172b54d000835e7d5
😎 Deploy Preview https://deploy-preview-4--musical-cucurucho-b52132.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@NachoBasilio NachoBasilio mentioned this pull request May 11, 2026
3 tasks done
@NachoBasilio
NachoBasilio merged commit fa28023 into dev May 11, 2026
5 checks passed
@NachoBasilio
NachoBasilio deleted the feat/catalogo-limpio branch May 11, 2026 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant