Repository navigation
fix(tipos): renombrar uno de fábrica ahora se ve en toda la aplicación - #89
Closed
JoseSerpaCode wants to merge 1 commit into
Closed
JoseSerpaCode wants to merge 1 commit into
JoseSerpaCode wants to merge 1 commit into
Conversation
La regla que decide qué nombre enseñar estaba copiada cinco veces, y las cinco ignoraban el nombre escrito cuando el tipo era de fábrica: ganaba siempre la traducción de la clave. Renombrar «Task» a «Incidencia» solo se veía al reabrir el diálogo de renombrar, que lo lee de un `data-name`. Se traduce mientras el tipo esté intacto y gana el nombre propio en cuanto alguien lo cambia — la misma regla que el proyecto ya aplica a etiquetas y a tipos propios. Compara contra el nombre con el que nació, para no quitarle la traducción a quien nunca renombra. Las cinco copias pasan a `nombreVisible()`, junto a `mapaPorClave()`. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Owner
Author
|
Consolidada en una sola PR con las cinco correcciones y una versión coherente. Se abrió como |
3 tasks done
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.
A.1 del plan — el más engañoso de los siete, porque parece que guarda bien.
issueTypes.editar()sí guardaba el nombre. El fallo estaba en los cinco sitios que lo pintan, todos con la misma expresión copiada:Para un tipo de fábrica eso ignora
ti.namesiempre. El único sitio donde aparecía el nombre nuevo era el diálogo de renombrar, que lo lee de undata-name— de ahí el reporte: «si lo renombro otra vez sí aparece, pero en el resto de la web sigue el anterior».La regla
La que el proyecto ya aplica a etiquetas y a tipos propios: lo que escribe una persona no se traduce.
Se decide comparando contra el nombre con el que nació, no mirando solo
is_builtin; si no, se le quitaría la traducción a casi todo el mundo.Las cinco copias pasan a
nombreVisible(), junto amapaPorClave()que ya existía para esto mismo.Plan de pruebas
npm run lint,npm run typecheck— limpios🤖 Generated with Claude Code