Repository navigation
Cognoms catalans de cada lloc i l'inici sobre el feltre - #50
Conversation
- nomsLliga.js tornat a copiar d'El Tauler: cognoms catalans més estesos del País Valencià; a les Illes, els de cada illa (Mallorca, Menorca i les Pitiüses); a la Catalunya Nord, en català; a l'Alguer, els catalans que s'hi conserven, amb els sards com a gent d'altres orígens. - Inici: les tres targetes amb el feltre de la taula de joc de fons i el jugador a dalt a l'esquerra. 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: df0d996f03
ℹ️ 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 (!where) { | ||
| const island = fold(home && home.comarca); | ||
| if (island === 'menorca') return 'menorca'; | ||
| if (island === 'eivissa' || island === 'formentera') return 'pitiuses'; |
There was a problem hiding this comment.
When a player enters an unlisted locality and selects Menorca or Eivissa/Formentera in LocationDialog, local rivals incorrectly receive Mallorcan names. For example, normalizeHome({ territory: 'bal', municipi: 'Fornells', comarca: 'Menorca' }) is accepted by the location dialog, but placeCulture('bal', '', home) returns 'bal'. Because a normalized home always has a nonempty municipi, where is truthy and the comarca fallback never runs. When the rival has no origin and the home locality is not recognized, use the explicitly selected island instead of defaulting to Mallorca.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Cert: sense origen del rival, placeCulture mira primer home.municipi, que sempre hi és; un poble que no és a la llista (Fornells, a Menorca) cau a Mallorca i l'illa triada (home.comarca) no s'arriba a mirar. La correcció és fer servir l'illa de home.comarca quan el rival no té origen, i el municipi només si no hi ha illa (també a nomslliga.js d'El Tauler, d'on es copia). Va al proper canvi.
Generated by Claude Code
Què canvia
nomsLliga.js, tornat a copiar d'El Tauler), també a les lligues locals:placeCulture): Mallorca (Pons, Ferrer, Vidal, Coll, Riera, Oliver, Bauzà…), Menorca (Pons, Seguí, Mercadal, Carreras, Sintes, Taltavull…) i les Pitiüses (Marí, Tur, Ribas, Torres, Prats, Planells…), pel lloc del rival o, si no se sap, del jugador..board), amb els colors de cada mode en la versió clara; el jugador a dalt a l'esquerra i el nom del joc a la dreta.Proves
npm run typecheckinpm testen verd (176 + 233). Test nou: a la lliga local de Palma, Maó, Eivissa, Perpinyà, l'Alguer i Gandia, els cognoms són els del lloc.Generated by Claude Code