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.

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

[#2809, #2858, #2859] Add user configuration for status conditions #2846

Merged

Conversation

krbz999
Copy link
Contributor

@krbz999 krbz999 commented Feb 3, 2024

Adding in some much needed user agency, a quick one-liner can now modify which condition does what.

Doesn't touch exhaustion.

Closes #2809.
Closes #2858.
Closes #2859.

@krbz999 krbz999 changed the title [#2795, #2809] Add user configuration for status conditions and exhaustion [#2795, #2809, #2858, #2859] Add user configuration for status conditions and exhaustion Feb 3, 2024
@arbron
Copy link
Collaborator

arbron commented Feb 3, 2024

Would you be able to split the exhaustion changes from the other condition changes? I have a thought on the whole exhaustion design that I'd like to riff on that might take a bit longer but it would be good to get the other changes in the next version.

@krbz999
Copy link
Contributor Author

krbz999 commented Feb 3, 2024

I have a thought on the whole exhaustion design that I'd like to riff on

Would it conflict?

@krbz999 krbz999 changed the title [#2795, #2809, #2858, #2859] Add user configuration for status conditions and exhaustion [#2809, #2858, #2859] Add user configuration for status conditions and exhaustion Feb 3, 2024
@arbron arbron added this to the D&D5E 3.0.2 milestone Feb 4, 2024
@arbron arbron added the api label Feb 4, 2024
module/documents/actor/actor.mjs Outdated Show resolved Hide resolved
module/config.mjs Outdated Show resolved Hide resolved
@krbz999 krbz999 requested a review from arbron February 4, 2024 01:30
@krbz999 krbz999 requested a review from arbron February 4, 2024 18:54
@arbron arbron merged commit 0612a39 into foundryvtt:3.0.x Feb 6, 2024
@krbz999 krbz999 deleted the configurable-status-condition-properties branch February 6, 2024 20:03
@krbz999 krbz999 changed the title [#2809, #2858, #2859] Add user configuration for status conditions and exhaustion [#2809, #2858, #2859] Add user configuration for status conditions Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment