Skip to content

XPHB feats foundry integration - #17

Draft
alterNERDtive wants to merge 1 commit into
5etools-mirror-3:mainfrom
alterNERDtive:foundry-integrations
Draft

alterNERDtive wants to merge 1 commit into
5etools-mirror-3:mainfrom
alterNERDtive:foundry-integrations

Conversation

@alterNERDtive

Copy link
Copy Markdown

WIP Foundry integration for 2024 stuff.

I’m mostly interested in knowing if this style of doing things is OK; specifically the Tavern Brawler implementation as an Enchantment for Unarmed Strike.

@alterNERDtive

Copy link
Copy Markdown
Author

Speaking of Tavern Brawler … is there a way to make it automatically import Unarmed Strike as well?

@TheGiddyLimit

Copy link
Copy Markdown
Collaborator

looks sensible enough to me; my one teeny tiny nit-pick would be to avoid all-caps (e.g. ONLY) in activity names, and stick to title-case

is there a way to make it automatically import Unarmed Strike as well

afraid not, no

@TheGiddyLimit

Copy link
Copy Markdown
Collaborator

the other thing is that I've yet to purloin XPHB Foundry stuff, so some of this may end up being duplicate work

@alterNERDtive

Copy link
Copy Markdown
Author

the other thing is that I've yet to purloin XPHB Foundry stuff, so some of this may end up being duplicate work

Well some of that is pretty gross, like their GWM implementation.

@TheGiddyLimit

Copy link
Copy Markdown
Collaborator

it may be gross, but it is free!

@alterNERDtive

Copy link
Copy Markdown
Author

Well to be completely honest the entire point of this exercise is to make things less gross to me ;)

@alterNERDtive
alterNERDtive force-pushed the foundry-integrations branch from 640a173 to 9edc858 Compare May 22, 2025 10:01
@TheGiddyLimit

Copy link
Copy Markdown
Collaborator

@alterNERDtive XPHB stuff for feats is now in (though not yet in Plutonium, as that update's still WIP), if you want to merge/apply your better versions over the top

@alterNERDtive

Copy link
Copy Markdown
Author

I have a lot on my plate right now, so might be a bit. But good to know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants