Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@mvilanova
Copy link
Contributor

@mvilanova mvilanova commented May 17, 2023

This PR adds a new section to the project settings for viewing and deleting search filters. Options for creating new and editing existing filters will be added in the future. We only allow users with admin, manager, and owner role to delete filters.

Screenshot 2023-05-17 at 5 52 32 PM Screenshot 2023-05-17 at 5 54 37 PM

@mvilanova mvilanova added the enhancement New feature or request label May 17, 2023
@mvilanova mvilanova requested review from kevgliss and wssheldon May 17, 2023 21:52
@kevgliss
Copy link
Contributor

Do we want to add more permissions to these? I think they drive quite a lot of functionality, creating new filters is probably fine. But deleting/modifying/disabling existing filters is likely to be problematic.

@mvilanova
Copy link
Contributor Author

mvilanova commented May 18, 2023

Do we want to add more permissions to these? I think they drive quite a lot of functionality, creating new filters is probably fine. But deleting/modifying/disabling existing filters is likely to be problematic.

Yeah, I can add permissions in a separate PR. DONE

@mvilanova mvilanova force-pushed the enhancement/search-filter-ui branch from 6f1b23e to 0d44e2f Compare May 18, 2023 00:40
@mvilanova mvilanova merged commit 313398e into master May 18, 2023
@mvilanova mvilanova deleted the enhancement/search-filter-ui branch May 18, 2023 01:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants