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

Mark Loot Sheet NPC 5E as dependency so that the Module Management screen will track it #48

Closed
TheRealIceCracker opened this issue Aug 6, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@TheRealIceCracker
Copy link

Is your feature request related to a problem? Please describe.
Loot Sheet NPC 5E isn't marked as a dependency, so the Module Management screen won't notify you if you don't have that installed as well. This renders this module non-functional with no clear in-app indication why.

Describe the solution you'd like
Loot Sheet NPC 5E marked as a dependency such that the Module Management screen will notify you if you don't have that installed or enabled.

Describe alternatives you've considered
At the end of the day I figured it out just by reading the module description again, but most other modules have this functionality and I assume (please don't crucify me if it's not) that it's pretty low lift.

Additional context
N/A

@TheRealIceCracker TheRealIceCracker added the enhancement New feature or request label Aug 6, 2023
@DFreds
Copy link
Owner

DFreds commented Dec 18, 2023

The whole thing needs a rewrite, and might not rely on any loot sheet when its done. Closing this for now

@DFreds DFreds closed this as completed Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants