[Cases] Fix stale submitting#244543

Merged

lgestc merged 3 commits intoelastic:mainfrom

lgestc:fix_stale_submitting

Dec 1, 2025

Contributor

lgestc commented Nov 27, 2025 •

edited by kibanamachine

Loading

Summary

Addresses #167756

In short, whenever an error is thrown in the submit method, we should reset submission state, allowing users to retry.
Currently, an error thrown in .submit locks isSubmitting in false state.

Checklist

Unit or functional tests were updated or added to match the most common scenarios

lgestc requested review from a team as code owners

November 27, 2025 15:06

lgestc added v9.3.0 release_note:fix backport:all-open labels

lgestc force-pushed the fix_stale_submitting branch 2 times, most recently from f08b68c to 9c69585 Compare

November 27, 2025 15:08

Contributor

janmonschke commented Nov 27, 2025

@lgestc Could you add a unit test for the new behaviour?

Contributor Author

lgestc commented Nov 27, 2025

@lgestc Could you add a unit test for the new behaviour?

working on it

lgestc force-pushed the fix_stale_submitting branch 2 times, most recently from 40da4f7 to 482d376 Compare

November 27, 2025 16:28

lgestc commented

View reviewed changes

x-pack/platform/plugins/shared/cases/public/components/create/use_submit_case.tsx

+                    ]
+                  ),
+                  isSubmitting:
+                    isPostingCase || isCreatingAttachments || isPostingObservables || isPushingToExternalService,

Contributor Author

lgestc Nov 27, 2025

this is important

lgestc force-pushed the fix_stale_submitting branch 3 times, most recently from b2e0725 to 5f383b9 Compare

November 27, 2025 16:36

lgestc changed the title ~~[Shared][Forms] Fix stale submitting~~

lgestc added the Team:Cases label

Contributor

elasticmachine commented Nov 27, 2025

Pinging @elastic/kibana-cases (Team:Cases)

darnautov removed the request for review from a team

November 28, 2025 09:47

lgestc force-pushed the fix_stale_submitting branch 3 times, most recently from 7759a00 to 1fef8db Compare

November 28, 2025 12:35


          [Case] Fix stale submitting state

58ed2a0

lgestc force-pushed the fix_stale_submitting branch from 1fef8db to 58ed2a0 Compare

November 28, 2025 15:02


          fix test

1e523ed

janmonschke approved these changes

View reviewed changes

Contributor

janmonschke left a comment

Thanks for fixing this long-standing bug 🎉


          Merge branch 'main' into fix_stale_submitting

2a4fff5

elastic deleted a comment from elasticmachine

Contributor

kibanamachine commented Jan 16, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

29 similar comments

Contributor

kibanamachine commented Jan 19, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 20, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 21, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 22, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 23, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 26, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 27, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 28, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 29, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Jan 30, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 2, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 3, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 4, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 5, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 6, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 9, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 10, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 11, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 12, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 13, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 16, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 17, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 18, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 20, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 23, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 24, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 25, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 26, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

Contributor

kibanamachine commented Feb 27, 2026

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 244543 locally
cc: @lgestc

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

Labels

backport:all-open backport missing release_note:fix Team:Cases v9.3.0