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

FeatureRequest: It should be possible to edit/clear the Spell List Registry #4946

Open
jennis0 opened this issue Dec 31, 2024 · 0 comments
Open

Comments

@jennis0
Copy link

jennis0 commented Dec 31, 2024

Currently it is not possible to remove spell lists which have been added to the Registry as all the underlying storage fields are marked private and there is no public API for doing so.

This causes issues for modules that want to modify and/or replace spell lists which are loaded by other modules (e.g. replacing the Spell LIsts from Tasha's with ones which reference the 2024 editions of spells).

In general, I think this would be valuable functionality for all similar registries.

Edit to add the more specific benefits I think this would bring:

Context: I'm building a module which will allow DMs to better manage what character options are available to their players via the Compendium Browser. The key thing is that it (a) doesn't rely on the player inputting the right set of filters and (b) it is more granular than just compendiums, so individual subclasses, feats, etc. can be approved or banned. I don't want to get that specific on spells, but do at least want to be able to control which spell list pages to include.

Benefits to allowing the removal of spell lists added by modules would be:

  • It allow DMs to remove or replace individual spells in modules (e.g. in editing Conjure Minor Elementals in PHB 2024)
  • It allows control on spell list source, not just spell source - e.g. Tasha's spell lists have only been partially replaced, so I do want to include Tasha's spells, but currently that leaves players with duplicates in the browser. Removing the spell lists means I can then create new spell lists that include the remaining spells but not the default ones
  • It creates a much cleaner player experience than relying on filters - no duplicates, no having to remember correct source, etc. ( I have quite a large and well-sourced set of content and I really really don't want to have to give my players a list of books and say - every time you want to look at spells, remember to select these 7 filters so you see the approved options.)

Desired outcome: Either make the byType/bySpell/loading properties of SpellListRegistry public or provide a function to reset the spell list. Slightly more complex functionality to remove a specific spell might also be valuable.

@jennis0 jennis0 changed the title FeatureRequest: It should be possible to remove/clear the Spell List Registry FeatureRequest: It should be possible to edit/clear the Spell List Registry Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants