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

Surfacing resource-related quick starts #530

Merged
merged 3 commits into from May 21, 2021

Conversation

gdoyle1
Copy link
Contributor

@gdoyle1 gdoyle1 commented Apr 6, 2021

This PR describes how we should be surfacing quick starts on resource pages, whether they are populated or empty. There is not a timeline of when this will get implemented.

I added some future design considerations at the end since I know other getting started efforts are going on at the same time. We have already discussed this @lwrigh so I tried pulling in the ideas you had there!

@bmignano @itsptk if you don't mind reviewing this to make sure it's up-to-date with quick starts and operator quick starts, that'd be awesome!

@openshift/team-ux-leads
@openshift/team-ux-review (Administrator perspective)
@openshift/team-devconsole-ux (Developer perspective)
@openshift/team-kni-ux (KNI & Virtualization)

@beaumorley
Copy link
Contributor

Added a few comments about the copy. Design looks good to me. We looked at this design today in the pipelines UX meeting. When building something like a pipeline is a time when a more in context approach is nice.

Copy link
Contributor

@itsptk itsptk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks awesome, thanks for capturing. I intend to add an operator-related equivalent design this sprint and will reference this one.

@gdoyle1
Copy link
Contributor Author

gdoyle1 commented Apr 12, 2021

@beaumorley @itsptk Made some updates, if you could take another look, that'd be great!

Copy link
Contributor

@beaumorley beaumorley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@itsptk itsptk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice!

Copy link
Member

@beanh66 beanh66 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beanh66 beanh66 merged commit 0bc4afd into openshift:master May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants