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.

Sign up for GitHub

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

Jump to bottom

Fix parameter validation issues #97

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Fix parameter validation issues #97

merged 1 commit into from
Nov 10, 2023

Conversation

jkupferer
Copy link
Collaborator

  • Bug in parameter validation caused invalid parameter values to be set in status then allowed as if unchanged.

@jkupferer
Fix parameter validation issues
555fb2b 
- Bug in parameter validation caused invalid parameter values to be set
  in status then allowed as if unchanged.
tonykay
tonykay approved these changes Nov 10, 2023
View reviewed changes
Copy link

@tonykay tonykay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing stood out as wrong, but this is not 100% my domain.

@jkupferer jkupferer merged commit a977d90 into main Nov 10, 2023
1 check passed
@jkupferer jkupferer deleted the fix-parameter-validation branch November 10, 2023 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@perryrivera1 perryrivera1 perryrivera1 approved these changes

@tonykay tonykay tonykay approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jkupferer @tonykay @perryrivera1