mirror of
https://github.com/Unleash/unleash.git
synced 2025-10-27 11:02:16 +01:00
95a0c7748f
https://linear.app/unleash/issue/2-1165/improve-adminalert-usage-to-be-more-generic-accept-non-admin Upgrades our `AdminAlert` to a new `PermissionGuard`. **Question**: We don't **need** to, but **should** we be specific about the `ADMIN` permission every time? Technically `PermissionGuard` could have `permissions` as optional and assume `[]` by default, which will add `ADMIN` anyways. However, I feel like we may gain some readability if we're specific about it. WDYT? Single permission:  Multiple permissions:  |
||
|---|---|---|
| .. | ||
| apiToken | ||
| auth | ||
| billing | ||
| cors | ||
| groups | ||
| instance-admin | ||
| invoice | ||
| maintenance | ||
| menu | ||
| network | ||
| roles | ||
| serviceAccounts | ||
| users | ||
| Admin.tsx | ||
| AdminRedirect.tsx | ||
| LazyAdmin.tsx | ||
| LazyAdminExport.tsx | ||