Skip to content

Commit 3ea032c

Browse files
committed
chore: PR feedback tweaks
1 parent 8612e14 commit 3ea032c

7 files changed

+8
-13
lines changed

lib/utils/token/getFlag.ts

+3-6
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,11 @@ export const getFlag = async <T = string | boolean | number | object>(
1515
}
1616

1717
const flags = claims.feature_flags || claims["x-hasura-feature-flags"];
18-
console.log("flags", flags);
18+
1919
if (!flags) {
2020
return null;
2121
}
2222

23-
if (name) {
24-
const value = flags[name];
25-
return value ? (value?.v as T) : null;
26-
}
27-
return null;
23+
const value = flags[name];
24+
return (value?.v as T) ?? null;
2825
};

lib/utils/token/getFlagHasura.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ describe("getFlag - Hasura", () => {
1111
});
1212

1313
it("when no token", async () => {
14-
await storage.setSessionItem(StorageKeys.idToken, null);
14+
await storage.setSessionItem(StorageKeys.accessToken, null);
1515
const idToken = await getFlag("test");
1616

1717
expect(idToken).toStrictEqual(null);

lib/utils/token/getPermissionsHasura.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ describe("getPermissions - Hasura", () => {
1616
});
1717

1818
it("when no token", async () => {
19-
await storage.setSessionItem(StorageKeys.idToken, null);
19+
await storage.setSessionItem(StorageKeys.accessToken, null);
2020
const idToken = await getPermissions();
2121

2222
expect(idToken).toStrictEqual({

lib/utils/token/getUserOrganistaions.test.ts lib/utils/token/getUserOrganizations.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ describe("getUserOrganizations", () => {
1010
setActiveStorage(storage);
1111
});
1212

13-
it("When single org", async () => {
13+
it("when token is null", async () => {
1414
await storage.setSessionItem(StorageKeys.idToken, null);
1515
const idToken = await getUserOrganizations();
1616

lib/utils/token/getUserOrganistaions.ts lib/utils/token/getUserOrganizations.ts

+1-3
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@ import { getDecodedToken } from ".";
55
* @returns { Promise<string[] | null> }
66
*/
77
export const getUserOrganizations = async (): Promise<string[] | null> => {
8-
const token = await getDecodedToken<{
9-
org_codes: string[];
10-
}>("idToken");
8+
const token = await getDecodedToken("idToken");
119

1210
if (!token) {
1311
return null;

lib/utils/token/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { getFlag } from "./getFlag";
88
import { getUserProfile, UserProfile } from "./getUserProfile";
99
import { getPermission, PermissionAccess } from "./getPermission";
1010
import { getPermissions, Permissions } from "./getPermissions";
11-
import { getUserOrganizations } from "./getUserOrganistaions";
11+
import { getUserOrganizations } from "./getUserOrganizations";
1212
import { getRoles } from "./getRoles";
1313
import { isAuthenticated } from "./isAuthenticated";
1414
import { refreshToken } from "./refreshToken";

0 commit comments

Comments
 (0)