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

[Web] Add resource kind-specific sidenav sections #47882

Merged
merged 2 commits into from
Oct 28, 2024

Conversation

rudream
Copy link
Contributor

@rudream rudream commented Oct 24, 2024

Purpose

Part of https://github.com/gravitational/teleport.e/issues/3998

This PR adds resource kind-specific sidenav shortcuts, allowing users to quickly filter for resource kinds as well as pinned resources from the sidenav. They have also been added to the search.

For more details and context on the way this interacts with the user preference for pinned resources, see the slack discussion here.

Demo

resources-sidenav.mov

@rudream rudream added the no-changelog Indicates that a PR does not require a changelog entry label Oct 24, 2024
@github-actions github-actions bot requested review from avatus and kiosion October 24, 2024 08:25
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47882.d212ksyjt6y4yg.amplifyapp.com

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47882.d3pp5qlev8mo18.amplifyapp.com

@rudream rudream requested a review from gzdunek October 24, 2024 10:37
@rudream rudream force-pushed the yassine/side-nav-resource-types branch from 7b5520c to 488e412 Compare October 24, 2024 11:32
@rudream rudream requested a review from gzdunek October 24, 2024 11:32
@rudream rudream force-pushed the yassine/side-nav-resource-types branch from 488e412 to ba9e9a7 Compare October 24, 2024 19:45
Copy link
Contributor

@avatus avatus left a comment

Choose a reason for hiding this comment

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

After gz's feedback is addressed, LGTM

Copy link
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

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

Leaving a preemptive approval.

@rudream rudream force-pushed the yassine/side-nav-resource-types branch from ba9e9a7 to 20d783b Compare October 28, 2024 06:57
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from kiosion October 28, 2024 06:57
@rudream rudream force-pushed the yassine/side-nav-resource-types branch from 20d783b to 3332972 Compare October 28, 2024 06:58
@rudream rudream enabled auto-merge October 28, 2024 06:59
@rudream rudream disabled auto-merge October 28, 2024 07:03
@rudream rudream force-pushed the yassine/side-nav-resource-types branch 2 times, most recently from f84f607 to a4e916a Compare October 28, 2024 07:06
@rudream rudream enabled auto-merge October 28, 2024 07:07
@rudream rudream force-pushed the yassine/side-nav-resource-types branch from a4e916a to bc98e26 Compare October 28, 2024 07:20
@rudream rudream disabled auto-merge October 28, 2024 07:20
@rudream rudream enabled auto-merge October 28, 2024 07:20
@rudream rudream added this pull request to the merge queue Oct 28, 2024
Merged via the queue into master with commit dbf356b Oct 28, 2024
42 checks passed
@rudream rudream deleted the yassine/side-nav-resource-types branch October 28, 2024 07:56
rudream added a commit that referenced this pull request Oct 28, 2024
* add resource kind-specific nav sections

* update eref
rudream added a commit that referenced this pull request Oct 28, 2024
* sidenav related fixes and tweaks (#47889)

* add resource kind-specific sidenav sections (#47882)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a PR does not require a changelog entry size/md ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants