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.

Sign up for GitHub

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

Jump to bottom

Add an enumeration for the potential states of the item attunement property. #882

Closed
aaclayton opened this issue Dec 13, 2020 · 2 comments
Closed

Add an enumeration for the potential states of the item attunement property. #882

aaclayton opened this issue Dec 13, 2020 · 2 comments
Assignees
@aaclayton
Labels
api
Milestone
D&D5E 1.2.x - Ong...

Comments

@aaclayton
Copy link
Contributor

Originally in GitLab by @tposney

It appears that there are no symbolic constants for attunement, meaning that checking the status of an item requires hard coding values like 0,1,2 to confirm the item status. This feels a little clumsy and it would be nice to have some CONFIG/CONST definitions for these, unless I have misunderstood the implementation.
@aaclayton
Copy link
Contributor Author

Fair point.

@aaclayton
Copy link
Contributor Author

mentioned in commit 836466e

@aaclayton aaclayton self-assigned this Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aaclayton aaclayton

Labels
api
Projects
None yet
Milestone
D&D5E 1.2.x - Ongoing Improvements
Development

No branches or pull requests
1 participant
@aaclayton