Skip to content

Comments

Add bravado trait to swashbuckler bravado actions and Strikes#20806

Merged
TikaelSol merged 3 commits intofoundryvtt:v13-devfrom
pedrogrullada:bravado-options
Feb 5, 2026
Merged

Add bravado trait to swashbuckler bravado actions and Strikes#20806
TikaelSol merged 3 commits intofoundryvtt:v13-devfrom
pedrogrullada:bravado-options

Conversation

@pedrogrullada
Copy link
Collaborator

@pedrogrullada pedrogrullada commented Nov 26, 2025

Also replace item:trait:bravado roll options with self:action:trait:bravado to be more accurate. Kept item:trait:bravado as that seems to be the preference, but I do need to account for self:action:trait:bravado in the Notes so it can catch Strikes that have had the trait added to them. I do think going with self:action:trait:bravado overall would be better in this case, but I'm deferring to the leads here.

Additionally, refresh Jirelle copies with these changes. Left Zathri alone as it's a pre-remaster pregen, no bravado for them.

@pedrogrullada pedrogrullada added pr: data update Updates to existing actors and items pr: data migration labels Nov 26, 2025
@pedrogrullada pedrogrullada force-pushed the bravado-options branch 2 times, most recently from fa1fe19 to 3aaa148 Compare December 2, 2025 15:38
@pedrogrullada pedrogrullada marked this pull request as draft January 3, 2026 18:50
@pedrogrullada pedrogrullada marked this pull request as ready for review February 5, 2026 15:01
@TikaelSol TikaelSol merged commit c7e37d0 into foundryvtt:v13-dev Feb 5, 2026
1 check passed
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.

2 participants