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
Add create project link in all the dev console pages #6258
Add create project link in all the dev console pages #6258
Conversation
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.
Hi @rohitkrai03 ! I see that we have the same text as +Add page in all the other views.What we wanted to do was just append the " or create a project" to the existing text on the other views. Eg: For Topology "Select a project to view the topology or create a project"
Agreed - original text should remain, and just append "or create a project" |
18e1064
to
315aaf6
Compare
@parvathyvr @serenamarie125 I actually didn't change text for any of the pages. If you look at the diff in code you'll see that I am just using a different component which adds that functionality. I just didn't realize that the component which was earlier written for +Add page was hard coded with its text. Thanks for catching that. I've updated the PR, take a look at the updated gif below - |
/retest |
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.
Thanks for updating @rohitkrai03! LGTM
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
verified changes locally
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: parvathyvr, rohitkrai03, sahil143 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Fixes: https://issues.redhat.com/browse/ODC-4448
Analysis / Root cause: Create project link was missing from project list page on Topology/Monitoring/Builds/Pipelines/Helm pages.
Solution Description: Use
CreateProjectListPage
instead ofProjectListPage
in all pages.Screen shots / Gifs for design review:
@openshift/team-devconsole-ux @parvathyvr
Browser conformance: