Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Kashrishi climbing feats #17114

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

CaptainWonders
Copy link

fixed kashrishi's scuttle-up and skilled climber feats to use selector, rather than type. Also made skilled climber change climb speed to movement speed when scuttle up is taken.

…r, rather than type. Also made skilled climber change climb speed to movement speed when scuttle up is taken.
Comment on lines 37 to 43
{
"key": "BaseSpeed",
"predicate": [
"feat:scuttle-up"
],
"selector": "climb",
"value": "@actor.attributes.speed.total"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This will need a toggle since it only applies when "climbing trees or cavern walls". I recommend you use a toggleable roll option predicated on feat:scuttle-up and then predicate this BaseSpeed on just that toggle.

Copy link
Author

Choose a reason for hiding this comment

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

Just committed a change for that

@pedrogrullada pedrogrullada added the pr: data update Updates to existing actors and items label Nov 1, 2024
@TikaelSol TikaelSol changed the title fixed Kashrishi climbing feats Fix Kashrishi climbing feats Nov 1, 2024
@TikaelSol
Copy link
Collaborator

PR titles should be in imperative case, starting with a capital letter and finishing the sentence If accepted this PR will __

},
{
"key": "RollOption",
"label": "Climbing Cavern Wall Or Tree",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please localize this in re-en.json, likely under something like PF2E.SpecificRule.Kashrishi.SkilledClimber.ClimbToggle

We try to keep plain text in rule elements as localization strings only.

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