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 1868070: Updates to content for quick starts #6307
Bug 1868070: Updates to content for quick starts #6307
Conversation
/assign rohitkrai03 |
/unassign |
Note that there is no icon associated with these quick starts. When the PR from @rohitkrai03 is merged, the default icon will be displayed. Here's animated gifs of the 3 quick starts: Gifs for design review: |
/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.
/lgtm
Verified locally. This is how it looks with changes from #6254 for default fallback icons.
/assign @christianvogt |
/retest |
/retitle Bug 1868070: Add 3 missing dev quick starts |
@serenamarie125: This pull request references Bugzilla bug 1868070, which is invalid:
Comment 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. |
/bugzilla refresh |
@abhi-kn: This pull request references Bugzilla bug 1868070, 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. |
/retest |
/retest Not sure I understand the cypress errors... but I'll keep an eye on what is failing. |
/retest |
23c04bf
to
4d66b6d
Compare
@serenamarie125 There appears to be conflicts -- please rebase on the latest Once you are confident that the content has gone through the last set of changes, please re-request review from the DevConsole team for the Thanks in advance! cc @rohitkrai03 |
96a9b9b
to
06b46ac
Compare
/retest |
/retest |
frontend/packages/console-app/src/components/quick-starts/data/add-healthchecks-quickstart.ts
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/quick-starts/data/monitor-sampleapp-quickstart.ts
Outdated
Show resolved
Hide resolved
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt, jerolimov, rohitkrai03, serenamarie125 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 |
@serenamarie125: The following test failed, say
Full PR test history. Your PR dashboard. 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 understand the commands that are listed here. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@serenamarie125: All pull requests linked via external trackers have merged: Bugzilla bug 1868070 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. |
For posterity, the Jira ticket is: https://issues.redhat.com/browse/ODC-4438 |
Updates the content of the 4 existing quick starts and add 3 missing dev quick starts:
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1868070