Repository navigation
Lligues dels Països Catalans sense noms castellans - #49
Conversation
- nomsLliga.js tornat a copiar d'El Tauler: la barreja d'altres orígens de cada territori ja no té noms castellans ni de l'Amèrica hispana (la seva part són noms del territori), i els noms valencians i de la Franja ja no tenen formes castellanes. - catalanRivals: una temporada començada amb noms castellans els canvia per catalans del territori en carregar-la (sempre els mateixos). - A Europa, al món i per a qui juga des de fora, els noms del país. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0135ikkUjsY7jDihSMn6EBta
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a8d950ce02
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (!isCastilianName(name)) { | ||
| return name.split(' ').map(function (w) { return CATALAN_FORMS[w] || w; }).join(' '); |
There was a problem hiding this comment.
When an existing season contains both Spanish names and territorial names needing conversion, generated replacements can collide with the later converted names. The conversion branch neither checks nor updates taken, which initially contains only the original names. For example, an old level-4 season generated with seed 6366, a Gandia home and 12 players contains both Iván Delgado and Amparo Calatayud; migration changes both to Fina Ribes. Reserve the converted names before generating replacements and check uniqueness across the final roster so different rivals remain distinguishable in standings and fixtures.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Cert: la branca de les formes castellanes (CATALAN_FORMS) ni comprova ni actualitza taken, i un nom nou pot coincidir amb una forma convertida més avall (Amparo Calatayud → Fina Ribes). Només passa en temporades d'abans que tinguin alhora un nom castellà i una forma castellana d'aquell territori, i que l'atzar triï justament aquella combinació. La correcció: primer convertir les formes de tota la llista i reservar-les a taken, i després generar els noms nous (també a nomslliga.js d'El Tauler, d'on es copia). Va al proper canvi.
Generated by Claude Code
| const season = validSeason(value.season); | ||
| // Els noms d'abans («· Grup B», «· Lliga B») amb el d'ara («· Divisió B»). | ||
| if (season) season.name = renameDivision(season.name); | ||
| if (season) season.rivals = catalanRivals(season, career.home); |
There was a problem hiding this comment.
For users upgrading with a league round already in progress, this renames the season roster but leaves the separately persisted game's names unchanged. In App.tsx, playRound resumes a matching session using saved.setup and saved.game directly, so the same rival now has a Catalan name in the standings but its old Spanish name and avatar during the resumed round. Migrate the saved game's opponent names and player names consistently with the season roster, without discarding the in-progress game.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Cert: una jornada de lliga a mig jugar d'abans del canvi es reprèn amb saved.setup i saved.game, i el rival hi conserva el nom d'abans fins que s'acaba aquella partida; el resultat va per identificador i la classificació ja surt amb el nom nou. Per sincronitzar-ho cal passar el mapa de noms canviats (catalanRivals) a la partida desada i reanomenar-hi els jugadors. Va al proper canvi.
Generated by Claude Code
Què canvia
nomsLliga.jstornat a copiar d'El Tauler (segueix/eltauler,nomslliga.js). De la lliga local als Països Catalans, des de qualsevol territori:catalanRivals(league.ts): una temporada començada amb noms castellans, en carregar-la, els canvia per noms catalans del territori, sense repetir nom ni cognom, i sempre els mateixos (llavor de la temporada). Els resultats van per identificador.Proves
npm run typecheckinpm testen verd (176 + 232).Generated by Claude Code