robert/ttrpg-initiative-tracker
1
0
Fork 0
Code Issues Pull Requests Releases 4 Wiki Activity

TODO: BUG-4 broader - all 5 activeDisplay setDoc calls clobber fields

Browse Source 
{merge:true} ignored by setDoc (replace per contract). Each write wipes
other fields on activeDisplay/status doc.
This commit is contained in:
david raistrick
2026-06-30 14:02:32 -04:00
parent b2fd06ed17
commit 49ea39ea93
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
TODO.md
+8
View File
@@ -60,6 +60,14 @@
duplicate id' — GREEN.
### BUG-4: hide-player-HP breaks display view (preexisting)
- **Broader than hide-HP**: ALL 5 `storage.setDoc(getPath.activeDisplay(), ...)` calls
use `{merge:true}` which is IGNORED (setDoc = replace per contract).
Each write clobbers other fields on activeDisplay/status doc.
- line 1619: hide-HP toggle → clobbers campaignId+encounterId (display paused)
- line 1648: start combat → clobbers hidePlayerHp
- line 1779: end combat → clobbers hidePlayerHp
- line 1997: deactivate → clobbers hidePlayerHp
- line 2002: activate → clobbers hidePlayerHp
- Toggle "hide player HP" in admin → display view flips to "Game Session Paused".
- Toggling back does NOT recover. Must re-activate encounter in encounters
panel to restore display.