Skip to content

Conversation

@Theroxenes
Copy link

Adds resistance rules elements to Inspire Heroics (Defense) spell effects.
Renames effects and labels to match remastered name Rallying Anthem.
Updates descriptions to reflect added rules element.

Adds resistance rules elements to Inspire Heroics (Defense) spell
effects.
Also renames them to match remastered Rallying Anthem, and updated
description to reflect added rules element.
@ammalagonc
Copy link
Collaborator

Rallying Anthem has a dedicated effect.

I believe we are keeping this effect as is for Legacy purposes (aka folks still using the Legacy version of the class. A data lead can potentially confirm or refute this.

@Theroxenes
Copy link
Author

Theroxenes commented Nov 14, 2025

Rallying Anthem has a dedicated effect.

I believe we are keeping this effect as is for Legacy purposes (aka folks still using the Legacy version of the class. A data lead can potentially confirm or refute this.

Thank you for the info, the motivation for creating this PR was that the Workbench macro for Fortissimo uses this spell effect but in that case it would probably be better to add a new effect there. The legacy Inspire Defense does provide resistance as well though.

"_id": "Chol7ExtoN2T36mP",
"img": "systems/pf2e/icons/spells/inspire-heroics.webp",
"name": "Spell Effect: Inspire Heroics (Defense, +2)",
"name": "Spell Effect: Inspire Heroics (Rallying, +2)",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"name": "Spell Effect: Inspire Heroics (Rallying, +2)",
"name": "Spell Effect: Inspire Heroics (Defense, +2)",

{
"key": "FlatModifier",
"label": "Inspire Defense",
"label": "Rallying Anthem",
Copy link
Collaborator

@ammalagonc ammalagonc Nov 14, 2025

Choose a reason for hiding this comment

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

Suggested change
"label": "Rallying Anthem",

"_id": "BKam63zT98iWMJH7",
"img": "systems/pf2e/icons/spells/inspire-heroics.webp",
"name": "Spell Effect: Inspire Heroics (Defense, +3)",
"name": "Spell Effect: Inspire Heroics (Rallying, +3)",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"name": "Spell Effect: Inspire Heroics (Rallying, +3)",
"name": "Spell Effect: Inspire Heroics (Defense, +3)",

{
"key": "FlatModifier",
"label": "Inspire Defense",
"label": "Rallying Anthem",
Copy link
Collaborator

@ammalagonc ammalagonc Nov 14, 2025

Choose a reason for hiding this comment

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

Suggested change
"label": "Rallying Anthem",

Copy link
Collaborator

Choose a reason for hiding this comment

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

We shouldn't bother with the labels at all. Not only are they unlocalized and we don't really need to add new keys for legacy effects, but the system will already strip the Effect prefix as well as the parentheticals.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah - fair point!

@ammalagonc ammalagonc added the pr: data update Updates to existing actors and items label Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: data update Updates to existing actors and items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants