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

🐛 Review form, fix onChange handlers #1839

Merged
merged 3 commits into from
Apr 15, 2024

Conversation

sjd78
Copy link
Member

@sjd78 sjd78 commented Apr 11, 2024

On the review form, fix the onChange handler for the following NumberInput fields:

  • criticality
  • priority

Note: This fix is a follow up to the fix in #1831. A grep across the codebase showed 2 more or the same problems.

Resolves: #1844

On the review form, fix the `onChange` handler for the
following `NumberInput` fields:
  - criticality
  - priority

Note: This fix is a follow up to the fix in konveyor#1831.  A grep
across the codebase showed 2 more or the same problems.

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
@sjd78 sjd78 modified the milestones: v0.4.0, v0.3.2 Apr 12, 2024
@sjd78 sjd78 added the cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch. label Apr 12, 2024
@sjd78 sjd78 merged commit 276bedd into konveyor:main Apr 15, 2024
7 checks passed
@sjd78 sjd78 deleted the fix-review-form-onChange branch April 15, 2024 17:29
github-actions bot pushed a commit that referenced this pull request Apr 15, 2024
On the review form, fix the `onChange` handler for the following
`NumberInput` fields:
  - criticality
  - priority

Note: This fix is a follow up to the fix in #1831. A grep across the
codebase showed 2 more or the same problems.

Resolves: #1844

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
Co-authored-by: Ian Bolton <ibolton@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>
sjd78 pushed a commit that referenced this pull request Apr 15, 2024
On the review form, fix the `onChange` handler for the following
`NumberInput` fields:
  - criticality
  - priority

Note: This fix is a follow up to the fix in #1831. A grep across the
codebase showed 2 more of the same problems.

Resolves: #1844
Backport-of: #1839

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
Co-authored-by: Ian Bolton <ibolton@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick/release-0.3 This PR should be cherry-picked to release-0.3 branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Cannot manually enter some values on the application review page
3 participants