Skip to content

[v17] Web: Add filters and pinning support for Enroll Resources page#52176

Merged
kimlisa merged 4 commits intobranch/v17from
lisa/v17/resource-select-filters
Feb 20, 2025
Merged

[v17] Web: Add filters and pinning support for Enroll Resources page#52176
kimlisa merged 4 commits intobranch/v17from
lisa/v17/resource-select-filters

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Feb 14, 2025

backports of #51698 #51672 #52017 #51893

the only conflict had to do with the pb.go files which were resolved with make grpc

changelog: Add filter drop-downs and pinning support for the "Enroll a New Resource" page in the web UI

* Web: Move resource specs into resource directory

* Web: move code out into separate files
…nce field (#51672)

* Add discover resource preferences to user preferences

* Define hard coded guide id consts

* Set guide ids for resources

* Add test

* Address CRs

* Define a new type just for SelectResource.tsx which requires id field
…52017)

This allows us to use "nil" value to mean no discover
guide preferences set (which is used to set default
values in the web UI).
* Minor changes

- Stop propagation and default behavior when clicking on pin button
- Add small or large icon option for discover icons

* Add pinning guide support

* Fix lint
@kimlisa kimlisa changed the title Web: Add filters and pinning support for Enroll Resources page [v17] Web: Add filters and pinning support for Enroll Resources page Feb 14, 2025
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from rudream February 18, 2025 14:14
@kimlisa kimlisa added this pull request to the merge queue Feb 20, 2025
Merged via the queue into branch/v17 with commit 65be383 Feb 20, 2025
@kimlisa kimlisa deleted the lisa/v17/resource-select-filters branch February 20, 2025 16:43
@fheinecke fheinecke mentioned this pull request Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants