mirror of
https://github.com/Unleash/unleash.git
synced 2025-10-18 11:14:57 +02:00
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: 
20 lines
509 B
TypeScript
20 lines
509 B
TypeScript
import { Navigate } from 'react-router-dom';
|
|
import useUiConfig from 'hooks/api/getters/useUiConfig/useUiConfig';
|
|
import { InvoiceAdminPage } from 'component/admin/invoice/InvoiceAdminPage';
|
|
|
|
const FlaggedBillingRedirect = () => {
|
|
const { uiConfig, loading } = useUiConfig();
|
|
|
|
if (loading) {
|
|
return null;
|
|
}
|
|
|
|
if (!uiConfig.flags.UNLEASH_CLOUD) {
|
|
return <InvoiceAdminPage />;
|
|
}
|
|
|
|
return <Navigate to="/admin/billing" replace />;
|
|
};
|
|
|
|
export default FlaggedBillingRedirect;
|