Larkinabout / fvtt-custom-dnd5e

11 stars 7 forks source link

[BUG] Cannot open Configure Spell School after saving the setting without filling the reference while adding a new school #105

Closed FreezinPenguin closed 1 month ago

FreezinPenguin commented 2 months ago

Describe the bug

When configuring a spell school, I accidentally saved and closed the setting while still adding a new spell school that is still missing reference (abbreviation, name, icons are all done, but haven't added any reference) and now I cannot open the Configure Spell Schools anymore.

This is the error from the browser console after clicking on the 'Configure Spell School' customdnd5e configurespellschool error

Module Data

Go into the module's settings, open up the Debug Settings menu, click Export Data, save and attach the file here.

Steps to reproduce

  1. Go to 'Custom D&D 5e'
  2. Click on 'Configure Spell Schools'
  3. Create a new spell school and fill out all the forms except reference
  4. Scroll down to 'Save and close'
  5. Try to open 'Configure Spell Schools' again

Versions:

Larkinabout commented 2 months ago

Can you attach your debug file please. There's a bug with the Spell Schools config that I'll fix with the next release, but I'll need to modify your debug file, so you can re-import it after you've updated.

FreezinPenguin commented 2 months ago

Gladly, will you be able to guide me on how to get the debug file? As I don't know where those are located.

Larkinabout commented 2 months ago

Certainly, go to Configure Settings > Custom D&D 5e. Scroll down to Debug Settings and open it. Click Export Data.

FreezinPenguin commented 2 months ago

custom-dnd5e.json

Hello! Sorry it took quite a while to get back to you. Here is the debug setting (DnD version is now 4.0.2 if that matters. Please let me know if I should revert back and give you the debug file for 3.3.1 instead).

Larkinabout commented 2 months ago

Here you go, import this using the Import Data option in the Debug Settings window: custom-dnd5e-FreezinPenguin.json

FreezinPenguin commented 1 month ago

I'm so sorry I totally forgot about this issue, but to update, yes it is fixed now! And thank you very much!