Unsaved DMN edits are lost when switching check editor tabs
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 55/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Ambito
- frontend
Direzione di ricerca
Start with CustomCheckDetail, KogitoDmnEditorView, and openDmnEditor to trace the DMN editor's mount and unmount behavior. Check how currentDmnModel and eligibilityCheck().dmnModel are updated when switching tabs or leaving the editor. Done means unsaved DMN edits survive tab switches, and leaving with unsaved changes warns the user where applicable.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
In the custom check editor, unsaved edits in the DMN editor are silently lost when the user switches to another tab and comes back.
Steps to reproduce
- Open a custom check and go to the DMN Definition tab.
- Edit the model. Don't click Save Changes.
- Switch to another tab, such as Testing, then back to DMN Definition.
Expected: the edits are still there, or the app warns before they are discarded.
Actual: the editor shows the last saved model and the edits are gone, with no warning.
Cause
CustomCheckDetail renders the tabs in a Switch/Match, so leaving the DMN tab unmounts KogitoDmnEditorView. When it remounts, openDmnEditor seeds the editor with the saved eligibilityCheck().dmnModel and calls onDmnModelChange with that model, which overwrites currentDmnModel (the only copy of the edits).
Leaving the check editor entirely (breadcrumbs, main navigation, closing the tab) also discards unsaved edits without a warning. The app has no useBeforeLeave or beforeunload guard.
Possible fixes
- Seed a remounted editor from
currentDmnModelwhen it differs from the saved model, so edits survive tab switches. - Or keep the editor mounted and hide it while other tabs are shown.
- Warn before leaving the page with unsaved edits (
useBeforeLeavefrom@solidjs/routerplusbeforeunload).
Related
Branch 523-clarify-check-renaming-and-streamline-the-publishing-workflow adds a warning on the Publish tab with a Save DMN edits button. That covers the Publish tab only, not the general loss on tab switches.
- Lingua principale
- Java
- Stelle
- 16
- Fork
- 5
- Merge medio
- 15h 50m
- PR unite (30g)
- 22
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CodeForPhilly/benefit-decision-toolkit
-
backend Good for newcomer quick win
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
CodeForPhilly/benefit-decision-toolkit#562 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation quick win
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
CodeForPhilly/benefit-decision-toolkit#445 ·
I maintainer di solito rispondono entro 1 giorno
-
Make issue templatesAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
CodeForPhilly/benefit-decision-toolkit#425 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
CodeForPhilly/benefit-decision-toolkit#564 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
CodeForPhilly/benefit-decision-toolkit#563 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CodeForPhilly/benefit-decision-toolkit
Issue simili
-
BC remittance upload: a claim MSP refused (S03) is saved with a blank status instead of refused (F)Apertatype: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
carlos-emr/carlos#4933 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
apache/rocketmq-dashboard#6925 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Vakore/ZappierGames#147 ·
-
Post 1.11.2 to SpigotMCApertaspigotmc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
Dans-Plugins/Wild-Pets#356 ·
I maintainer di solito rispondono entro 1 giorno
-
Post 1.16.2 to SpigotMCApertaspigotmc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
Dans-Plugins/Medieval-Roleplay-Engine#371 ·
I maintainer di solito rispondono entro 1 giorno