Skip to content

Add rules for verdant- and pristine- weapon feats#22362

Open
iDantar wants to merge 6 commits into
foundryvtt:v14-devfrom
iDantar:20964-add-automation-for-verdant-and-pristine-weapon-feats
Open

Add rules for verdant- and pristine- weapon feats#22362
iDantar wants to merge 6 commits into
foundryvtt:v14-devfrom
iDantar:20964-add-automation-for-verdant-and-pristine-weapon-feats

Conversation

@iDantar
Copy link
Copy Markdown
Collaborator

@iDantar iDantar commented May 18, 2026

P.S. not entirely sure about weapon groups predicates on Verdant Weapon - would appreciate the suggestions

@iDantar iDantar added pr: data update Updates to existing actors and items pf2e labels May 18, 2026
Comment thread static/lang/re-en.json Outdated
Comment thread packs/pf2e/feats/class/druid/level-1/verdant-weapon.json Outdated
iDantar and others added 2 commits May 27, 2026 18:47
I used icon's prompt as a base, but I like your suggestion more, thanks!

Co-authored-by: TikaelSol <80183198+TikaelSol@users.noreply.github.com>
@iDantar iDantar requested a review from TikaelSol May 27, 2026 21:22
"adjustName": false,
"choices": [
{
"label": "PF2E.SpecificRule.Exemplar.Ikon.Physical.Grant",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe make it a generic label since it's not limited to Exemplar anymore? And the other choice too.

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.

I think it's a good idea, but wouldn't we need a migration for that then?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess it's either that or making a duplicate, and creating a migration just to replace 1 string seems wasteful. Oh well, might well leave as is and migrate some other time together with other language keys replacement.

@iDantar
Copy link
Copy Markdown
Collaborator Author

iDantar commented May 28, 2026

Thought a bit more about it and moved to more general localization keys - you both were right

Comment thread static/lang/re-en.json
Comment on lines 6011 to 6019
"EquipmentVariety": "Select an item variety.",
"Existing": "Use an existing item in my inventory.",
"EvolutionFeat": "Select a first evolution feat.",
"Feat": "Select a feat.",
"Feature": "Select a feature.",
"FirstImplement": "Select a first implement.",
"Form": "Select a form.",
"Grant": "Grant me a new item.",
"Heritage": "Select a heritage.",
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.

These wouldn't go in here, as they aren't prompt text.

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.

Good call, moved them to PF2E.UI.RuleElements.GrantItem - feels appropriate to me, what would you say?

Comment thread packs/pf2e/feats/class/druid/level-1/verdant-weapon.json Outdated
iDantar and others added 2 commits June 1, 2026 02:49
Co-authored-by: TikaelSol <80183198+TikaelSol@users.noreply.github.com>
@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.

Missing precious metal selectors for Pristine Weapon druid feat

3 participants