fvtt-loot-sheet-npc-5e icon indicating copy to clipboard operation
fvtt-loot-sheet-npc-5e copied to clipboard

Published 20 hours ago • verified publisher icon jopeek

Fields Won't Update for Merchant Settings

Open LyricalStreetPoet opened this issue 3 years ago • 3 comments

Describe the bug On the settings tab of the merchant sheet, the fields reset to their original value after you change them. Once you do this once, they will work the next time.

To Reproduce Steps to reproduce the behavior:

  1. Create a Merchant Sheet
  2. Go to the settings tab
  3. Change any of the inventory settings.
  4. Tab away or hit enter.
  5. The value you entered will be gone and the prior value will show.

Expected behavior Save my changes the first time.

Screenshots image

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome 98.0.4758.82
  • Version: Lootsheet 3.5.0

LyricalStreetPoet avatar Feb 11 '22 22:02 LyricalStreetPoet

I'm having the same problem, although it doesn't always keep the changes the second time. Often I have to try and change it multiple times before it seems to stick. Also, it keeps reverting to Loot instead of Merchant.

donaldedgar avatar Feb 13 '22 10:02 donaldedgar

Please test if the problem persists in https://github.com/jopeek/fvtt-loot-sheet-npc-5e/releases/tag/v3.5.8 I suspect they will.

Cause so far I was not able to pin down the exact reason why the flags used for this data sometimes change as expected and sometime just 🤬🤬🤬🤬

I suspect that this is an issue with the asynchronous updates and the way foundry updates/reloads a changed sheet.

DanielBoettner avatar Feb 18 '22 23:02 DanielBoettner

As you suggested, the problem still persists. Just tested the latest version tonight in my regular campaign

LyricalStreetPoet avatar Feb 21 '22 04:02 LyricalStreetPoet