Skip to content

Commit a3fa5ea

Browse files
cezudasCopilot
andauthored
Fetch analytics project permissions from /me API instead of local storage (#1853)
Fixes OPS-3442. --------- Co-authored-by: Copilot <[email protected]>
1 parent 9b8cd4d commit a3fa5ea

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
1-
import { authenticationSession } from '@/app/lib/authentication-session';
21
import { FlagId } from '@openops/shared';
32
import { useMemo } from 'react';
43
import { flagsHooks } from './flags-hooks';
4+
import { userHooks } from './user-hooks';
55

66
export const useHasAnalyticsAccess = (): boolean => {
77
const { data: isAnalyticsEnabled } = flagsHooks.useFlag<boolean | undefined>(
88
FlagId.ANALYTICS_ENABLED,
99
);
1010

11-
const hasAnalyticsPrivileges =
12-
authenticationSession.getUserHasAnalyticsPrivileges();
11+
const { userMeta } = userHooks.useUserMeta();
12+
const hasAnalyticsAccess = userMeta?.projectPermissions?.analytics ?? false;
1313

1414
return useMemo(
15-
() => Boolean(isAnalyticsEnabled && hasAnalyticsPrivileges),
16-
[isAnalyticsEnabled, hasAnalyticsPrivileges],
15+
() => Boolean(isAnalyticsEnabled && hasAnalyticsAccess),
16+
[isAnalyticsEnabled, hasAnalyticsAccess],
1717
);
1818
};

packages/react-ui/src/app/lib/authentication-session.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,6 @@ export const authenticationSession = {
5151
return this.getCurrentUser()?.projectRole ?? null;
5252
},
5353

54-
getUserHasAnalyticsPrivileges(): boolean {
55-
return this.getCurrentUser()?.hasAnalyticsPrivileges ?? false;
56-
},
57-
5854
getUserOrganizationRole() {
5955
return this.getCurrentUser()?.organizationRole ?? null;
6056
},

0 commit comments

Comments
 (0)