advancement-wip

Nice work!

I NEED DIS

btw the "Class Restriction" thingy looks weird to me.. this is inside the class right ? what is that All Classes supposed to mean ? like, it's not affected by subclass or something like that ?

It's for restrictions against multiclassing

hmm I see. I don't find that "All Classes" really that clear though, maybe we could explore other options

more hints never hurt anyone, is good feedback

very interesting are you willing to integrated some mechanism to choose the feature like in this old module ? https://github.com/ZoltanTheDM/class-editor

if im not misunderstanding both Advancements and class-editor module, they literally do the same. Only Advancements will be a 5e system feature instead of a module.

hey Calego, would you say its moderately safe for me to already start playing with that ? right now im doing heavy gymnastics to get the class features on HCT, and I will need to reimplement big chunks of that from scratch for Advancements, so the earlier I can play around with an implementation the better 😆

@ccjmk the API is still in flux but absolutely.
Make a new world and remember to switch your cloned 5e repo back to master for normal stuff

oh, I still need your pending MR to add items to advancements, right ?

and I assume HP is a placeholder for now, other than been auto added too all levels.

hmm I guess I could set your repo as another remote and just pull your latest commits from the MR branch 🤔

ehh maybe I'll wait until your MR gets merged 😛