-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
Add 'proficiency' property to subclass object #258
Comments
A better solution imo would be to expand on the "Bonus Proficiency" feature, rather than the subclass itself. According to the SRD, the feature is what grants the proficiency, not the subclass. Features in general could definitely do with being expanded to make them more computer friendly - e.g. using APIReferences, Choice structures, and so on. |
hmm, so Also, I agree on the idea of expanding the features. For example, it would be nice if features like "Divine Domain' or "Bard College" had some list property to choose from (despite the fact there is only one subclass choice for every class). Sometimes, to get to actual data related to a feature is tricky |
Yes, exactly this! It would be better representative of the SRD content, and just meshes better with the semantics of the API data's structure as a result. |
This is similar to #187 as well. |
So
When you choose life domain for cleric, you get proficiency with heavy armor, but it is not specified in data itself, you just get a feature called "bonus proficiency", which itself is not too descriptive.
My suggestion is, maybe add
proficiencies
property to domain object? And add such property to aby subclass giving you proficiencies?The text was updated successfully, but these errors were encountered: