Repository navigation
El nombre de quien atendió pasa al pie del ticket - #33
Merged
Merged
Conversation
Pedido de David 2026-08-20: quitar la columna vertical del margen derecho (V A L E R I A hacia abajo, del 2026-08-11) y poner el nombre abajo, en el renglon que ocupaba "!Vuelva pronto!". El nombre sale completo y centrado en negritas, no letra por renglon, asi que ya no hace falta recortarlo al nombre de pila ni quitarle acentos por espacio: es el mismo texto que ya salia en la cabecera. Las 4 columnas del margen vuelven al ticket. No se devolvieron a las columnas de dinero —que siguen en 6 y sobradas: el importe mas alto jamas cobrado en un renglon fue $800— sino al nombre del platillo, que pasa de 32 a 36 columnas y se corta menos que en cualquiera de los dos estados anteriores. Sin mesera el pie se queda con la despedida de siempre, para que el ticket no acabe en seco. Los tests del margen vertical se reemplazan por los del pie, y el ayudante margenDerechoDe se queda como centinela: ahora afirma que la orilla derecha esta vacia. Comprobado mutando el codigo (al volver a "!Vuelva pronto!" el test falla) y mirando el dibujo con VistaPreviaTicket.
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.
Pedido de David (2026-08-20): quitar el nombre de la mesera que bajaba por el margen derecho y ponerlo abajo, en el renglón que decía !Vuelva pronto!.
Qué cambia
Antes (desde el 2026-08-11) / ahora:
Verificación
./mvnw test: 95 tests en verde. (RestApiApplicationTestsfalla en local por falta deDB_USER/DB_PASSWORD; en el CI, que levanta MySQL vacío, corre.)margenDerechoDese queda como centinela: ahora afirma que la orilla derecha está vacía, así que si la columna vertical volviera, el test lo caza."!Vuelva pronto!",elPieTraeElNombrefalla. El test sabe fallar.VistaPreviaTicketen los 4 casos (nombre normal, nombre largo, platillo que no cabe, sin mesera) antes de abrir el PR.Falta la prueba en papel de la térmica — la misma que quedó pendiente del #32.