Skip to content

Clear forms on save#1922

Merged
rupurt merged 2 commits into169601756-null-persist-url-errorfrom
chore/clear-forms-on-save
Nov 12, 2019
Merged

Clear forms on save#1922
rupurt merged 2 commits into169601756-null-persist-url-errorfrom
chore/clear-forms-on-save

Conversation

@rupurt
Copy link
Copy Markdown
Contributor

@rupurt rupurt commented Nov 12, 2019

Improve bridge & job spec form persistence. Extracted from #1876 as it addresses a different problem to the underlying story.

  • Save form input on error
  • Clear saved input on success
  • Show changed form nav warning when different to initial or persisted

[Fixes #169698197]

@rupurt rupurt force-pushed the 169601756-null-persist-url-error branch from 478c8af to c94a261 Compare November 12, 2019 07:10
@rupurt rupurt force-pushed the chore/clear-forms-on-save branch from ff17e7f to 5c5cafe Compare November 12, 2019 07:12
Alex Kwiatkowski added 2 commits November 11, 2019 23:17
- Save form input on error
- Clear saved input on success
- Show changed form nav warning when different to initial or persisted
- Save form input on error
- Clear saved input on success
- Show changed form nav warning when different to initial or persisted
@rupurt rupurt force-pushed the chore/clear-forms-on-save branch from 5c5cafe to e51b342 Compare November 12, 2019 07:17
@rupurt rupurt merged commit bd5bb31 into 169601756-null-persist-url-error Nov 12, 2019
@rupurt rupurt deleted the chore/clear-forms-on-save branch November 12, 2019 18:38
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