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
Bug 1830046: Helm redirects based on the origin of the action #5254
Bug 1830046: Helm redirects based on the origin of the action #5254
Conversation
@rohitkrai03: This pull request references Bugzilla bug 1830046, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@rohitkrai03: This pull request references Bugzilla bug 1830046, which is valid. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
1 similar comment
@rohitkrai03: This pull request references Bugzilla bug 1830046, which is valid. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
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.
LGTM..Thanks!
verified locally and works fine. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: beaumorley, karthikjeeyar, parvathyvr, rohitkrai03 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 |
@rohitkrai03: All pull requests linked via external trackers have merged: openshift/console#5254. Bugzilla bug 1830046 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
I find this approach sort of interesting. |
@christianvogt You mean instead of action caller explicitly sending origin context as a parameter, the action should figure out the current URL itself and send that to the form? Currently the default is helm list page for all actions but we can change that to details page. |
@rohitkrai03 |
@christianvogt I did think about using |
@rohitkrai03 we may need to revisit this later as we will want all actions to be contributed through extensions. We may still contribute separate actions for different views, but ideally the same action be used everywhere when possible. |
Fixes: https://issues.redhat.com/browse/ODC-3676
Analysis / Root cause: The helm action forms like upgrade or rollback were not designed to redirect based on context of the origin of the action.
Solution Description: Added origin context to helm actions which is then used by the forms to redirect to correct origin of that action.
This PR also includes small changes suggested by UX in final designs like the order and label of Helm actions. It also updates upgrade form description.
Screen shots / Gifs for design review:
cc: @openshift/team-devconsole-ux @parvathyvr
Rollback -
Upgrade -
Browser conformance: