Skip to content

fix: Reload Diagnostic Report after workflow action to resolve unsave…#960

Open
Irfan778899 wants to merge 1 commit into
earthians:version-16-hotfixfrom
Irfan778899:fix_save_diagnostic_report_after_results_approved
Open

fix: Reload Diagnostic Report after workflow action to resolve unsave…#960
Irfan778899 wants to merge 1 commit into
earthians:version-16-hotfixfrom
Irfan778899:fix_save_diagnostic_report_after_results_approved

Conversation

@Irfan778899

Copy link
Copy Markdown
Contributor

After performing a workflow action (e.g., Approve) on Diagnostic Report,
the backend updates observation status via set_observation_status, but
the form remains in a dirty (unsaved) state in the UI.

This causes the "Not Saved" indicator to appear even though the document
is already updated and shows correct status in list view.

Added frm.reload_doc() after successful workflow action to sync the form
state with backend and clear the false unsaved state.

Also added a safety check before calling save_action in before_save.

@Irfan778899 Irfan778899 changed the title fix: reload Diagnostic Report after workflow action to resolve unsave… fix: Reload Diagnostic Report after workflow action to resolve unsave… Apr 6, 2026
@Irfan778899 Irfan778899 force-pushed the fix_save_diagnostic_report_after_results_approved branch from 6292ba9 to cd5b1a8 Compare April 12, 2026 10:34
@Irfan778899 Irfan778899 force-pushed the fix_save_diagnostic_report_after_results_approved branch from cd5b1a8 to a405877 Compare April 12, 2026 10:42
@mujeerhashmi

Copy link
Copy Markdown
Contributor

@Sajinsr Kindly review this. Long pending. The users often get confused with this state of Not Saved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants