Skip to content

Localize family ability glossary#22386

Open
iDantar wants to merge 2 commits into
foundryvtt:v14-devfrom
iDantar:localize-bestiary-family-abilities
Open

Localize family ability glossary#22386
iDantar wants to merge 2 commits into
foundryvtt:v14-devfrom
iDantar:localize-bestiary-family-abilities

Conversation

@iDantar
Copy link
Copy Markdown
Collaborator

@iDantar iDantar commented May 22, 2026

Adds an ItemAlteration for name on each of the abilities in packs\pf2e\bestiary-family-ability-glossary, to replace it with a localized one, without parenthesis mentioning the family name. So (Vampire, Jiang-Shi, Basic) One More Breath becomes just One More Breath

To a kind reviewer: updates were done via script (included in PR for now, for convenience), so please don't be discouraged by change's size :)
One change post-script that I did was to Skymetal Metamorphosis abilities, as those already had entries in re-en

Kudos to @pedrogrullada for the idea

@iDantar iDantar added pr: data update Updates to existing actors and items pf2e labels May 22, 2026
Comment thread scripts/localize-family-glossary.mjs Outdated
Comment on lines +1 to +10
/**
* Apply bestiary-family-ability-glossary name localization for local testing.
*
* - Prepends an ItemAlteration rule that overrides each ability's display name
* - Adds PF2E.NPCAbility.<Family>.<Ability>.Name entries to static/lang/re-en.json
* - Ability keys use the pack name after the parenthetical prefix
* (e.g. "(Vampire, Jiang-Shi, Minister) Dark Enlightenment" → DarkEnlightenment)
*
* Usage: node scripts/localize-family-glossary.mjs [--dry-run]
*/
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the script?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thought this could be helpful for the review - not planning to include it in the final merge

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed it - it's still available in commit history, if needed

@iDantar iDantar requested a review from TikaelSol June 1, 2026 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pf2e pr: data update Updates to existing actors and items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants