Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enabled validation of determination draft saving #3707

Merged
merged 1 commit into from
Jan 8, 2024

Conversation

wes-otf
Copy link
Contributor

@wes-otf wes-otf commented Jan 3, 2024

Fixes #3706

A draft determination shouldn't be able to be saved without setting an outcome, as there isn't a valid state to store it in (and if a determination is being drafted, an outcome has been reached). This enforces partial form validation when a determination draft is saved to ensure an outcome is set.

Test Steps

  • Ensure user is a Staff role
  • Create or navigate to a Concept Note (ensure there is no determination made)
  • In the panel on the right, select Add Determination
  • Leave the Determination dropdown at -- No determination selected --
  • Attempt to select "Save draft" at the bottom of the page
  • Confirm Please select an item in the list is shown at the dropdown prompt, not allowing for a draft to be saved.

@wes-otf wes-otf added Type: Bug Bugs! Things that are broken :-/ Type: Patch Mini change, used in release drafter labels Jan 3, 2024
@wes-otf wes-otf requested review from theskumar and frjo January 3, 2024 21:15
@frjo frjo added the Status: Needs testing Tickets that need testing/qa label Jan 5, 2024
@frjo frjo merged commit fcd722e into main Jan 8, 2024
@frjo frjo deleted the fix/validate-determination-draft branch February 12, 2024 11:47
wes-otf added a commit that referenced this pull request May 7, 2024
Fixes #3706

A draft determination shouldn't be able to be saved without setting an
outcome, as there isn't a valid state to store it in (and if a
determination is being drafted, an outcome has been reached). This
enforces partial form validation when a determination draft is saved to
ensure an outcome is set.
wes-otf added a commit that referenced this pull request May 8, 2024
Fixes #3706

A draft determination shouldn't be able to be saved without setting an
outcome, as there isn't a valid state to store it in (and if a
determination is being drafted, an outcome has been reached). This
enforces partial form validation when a determination draft is saved to
ensure an outcome is set.
@frjo frjo removed the Status: Needs testing Tickets that need testing/qa label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bugs! Things that are broken :-/ Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Trying to save a draft determination without setting an outcome results in an error
2 participants