Repository navigation
feat: pestana Todos en el carrito y las vistas del DEV para el ADMIN - #14
Merged
Merged
Conversation
El buscador ya miraba todos los platillos, pero no habia forma de ver la carta completa sin teclear: las pestanas obligaban a elegir una categoria. Ahora Todos va al frente y no filtra nada. La mesa sigue abriendo en la primera categoria real, no en Todos: es el flujo que las seis tablets ya tienen aprendido. El filtro estaba copiado palabra por palabra en el carrito de mesas y en el de entregas; se saca a menuFiltro.js y las dos pantallas lo comparten, asi que el repartidor gana la misma pestana.
Le faltaba Platillos, la unica vista que era exclusiva de DEV. El dueno es quien cambia precios y da de alta platillos en la vida real, asi que tener que entrar como DEV para eso no tenia sentido. Los dos menus estaban escritos como listas separadas y habian divergido en silencio. Ahora es una sola lista para los dos roles: si algun dia uno necesita algo que el otro no, se separan a proposito y no por olvido. El permiso de verdad es el del backend y va en su propio PR de RestFoodB.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Va en pareja con RestFoodB #31. Merge los dos juntos: si entra solo este, el ADMIN ve la pantalla de Platillos y recibe 403 al guardar.
1. Pestaña "Todos" en el carrito
El buscador ya miraba todos los platillos —teclear "pozole" estando en Bebidas lo encontraba— pero no había forma de ver la carta completa sin teclear: las pestañas obligaban a elegir una categoría. Ahora "Todos" va al frente y no filtra nada.
La mesa sigue abriendo en la primera categoría real, no en "Todos": es el flujo que las seis tablets ya tienen aprendido, y abrir con 206 platillos de golpe es mucho scroll. "Todos" queda a un toque.
El filtro estaba copiado palabra por palabra en el carrito de mesas y en el de entregas. Se saca a
menuFiltro.jsy las dos pantallas lo comparten, así que el repartidor gana la misma pestaña sin código duplicado.2. El ADMIN ve lo mismo que el DEV
La única vista que era exclusiva de DEV era Platillos (alta, edición y borrado de la carta). Ya no.
Los dos menús estaban escritos como listas separadas y habían divergido en silencio. Ahora es una sola lista para los dos roles: si algún día uno necesita algo que el otro no, se separan a propósito y no por olvido.
Probado
npm run lintynpm run builden verde.Falta probarlo en tablet. Es un cambio de UI en la pantalla que más se usa, y en la computadora no se ve lo mismo: la fila de pestañas hace scroll horizontal en vertical y wrap en horizontal, así que "Todos" cambia dónde caen las demás. Vale la pena abrir una mesa en una tablet antes del merge.
Nota
El frontend sigue sin tener con qué correr tests (no hay vitest ni ningún runner).
filtrarMenuypestanasDeMenuquedaron como funciones puras justamente para que sean testeables el día que se monte. Montarlo es su propio PR.