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
Explicitly touch pipeline builder task fields when workspaces are changed #9130
Explicitly touch pipeline builder task fields when workspaces are changed #9130
Conversation
@rottencandy Even without this change I see the errors showing up correctly in the task sidbebar, I have verified it in the cluster latest build 4.8.0-0.nightly-2021-06-06-164529. cc: @jerolimov @andrewballantyne Edit: I have observed the above while creating a pipeline, but the while editing a Pipeline I see the issue. Thanks @rottencandy for this fix. |
frontend/packages/pipelines-plugin/src/components/pipelines/pipeline-builder/hooks.ts
Outdated
Show resolved
Hide resolved
1d072a3
to
ef17f90
Compare
/lgtm |
frontend/packages/pipelines-plugin/src/components/pipelines/pipeline-builder/hooks.ts
Outdated
Show resolved
Hide resolved
ef17f90
to
7c56e64
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the changed version again. Works fine.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewballantyne, jerolimov, karthikjeeyar, rottencandy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes:
https://issues.redhat.com/browse/ODC-5894
Analysis / Root cause:
Formik fields do not echo errors unless they are touched.
Solution Description:
Use a hook to explicitly touch all task fields so they echo errors when dependent fields are changed.
Screen shots / Gifs for design review:
0.mp4
Unit test coverage report:
(unchanged)
Test setup:
Browser conformance: