fix: la fecha cambiaba de formato al editarla, y «Cancelar» cerraba todo - #92
Open
JoseSerpaCode wants to merge 1 commit into
Open
fix: la fecha cambiaba de formato al editarla, y «Cancelar» cerraba todo#92JoseSerpaCode wants to merge 1 commit into
JoseSerpaCode wants to merge 1 commit into
Conversation
- El modal reescribe la fecha dentro de la propia tarjeta, y lo hacía con un `toLocaleDateString()` sin argumentos: el idioma del navegador en vez del de la aplicación. «9 sept 2026» pasaba a «9/09/26» en cuanto la tocabas. Es el fallo que la 1.25.0 corrigió en la tarjeta y del que quedó una copia aquí. - «Cancelar» en el alta de una base de datos hacía `close()`, exactamente lo mismo que la equis, en un flujo de dos pasos. Ahora deshace un paso. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
Dos de los tres hallazgos que reportaste. El tercero —lo primitivo del módulo de bases de datos— es una tanda de trabajo aparte y va al plan.
La fecha
La tarjeta la pinta el servidor con
fecha(), en el idioma de la aplicación: «9 sept 2026». Al cambiarla desde el modal, el script reescribe el texto dentro de la propia tarjeta con untoLocaleDateString()sin argumentos, que usa el idioma del navegador: «9/09/26».Es exactamente el fallo que la 1.25.0 corrigió para la tarjeta, del que quedó una copia suelta en el modal. Ahora los dos llaman a
fecha().El «Cancelar»
El alta tiene dos pasos —galería de plantillas y formulario— y el botón hacía
close(), igual que la equis. Cancelar en el segundo paso convertía «me he equivocado de plantilla» en «empieza de cero». Ahora deshace un paso y solo cierra desde la galería.Plan de pruebas
npm run lint,npm run typecheck— limpiosantes «Due: Sep 8, 2026» y después «Due: 9/9/2026», que reproduce el reporte🤖 Generated with Claude Code