fix(clerk-js): Persist "this" when revoking a session, from UserProfi…

…le (#4653)
clerk · Nov 25, 2024 · b85c5d7 · b85c5d7
1 parent 2d3866c
commit b85c5d7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/.changeset/mean-apples-approve.md b/.changeset/mean-apples-approve.md
@@ -0,0 +1,5 @@
+---
+'@clerk/clerk-js': patch
+---
+
+Bug fix: Being able to revoke a session from UserProfile.
diff --git a/packages/clerk-js/src/ui/components/UserProfile/ActiveDevicesSection.tsx b/packages/clerk-js/src/ui/components/UserProfile/ActiveDevicesSection.tsx
@@ -48,7 +48,7 @@ export const ActiveDevicesSection = () => {
 const DeviceItem = ({ session }: { session: SessionWithActivitiesResource }) => {
   const isCurrent = useSession().session?.id === session.id;
   const status = useLoadingStatus();
-  const [revokeSession] = useReverification(session.revoke);
+  const [revokeSession] = useReverification(session.revoke.bind(session));
 
   const revoke = async () => {
     if (isCurrent || !session) {

diff --git a/packages/clerk-js/src/ui/components/UserProfile/__tests__/SecurityPage.test.tsx b/packages/clerk-js/src/ui/components/UserProfile/__tests__/SecurityPage.test.tsx
@@ -113,6 +113,7 @@ describe('SecurityPage', () => {
             isMobile: false,
           },
           actor: null,
+          revoke: jest.fn().mockResolvedValue({}),
         } as any as SessionWithActivitiesResource,
         {
           pathRoot: '/me/sessions',
@@ -131,6 +132,7 @@ describe('SecurityPage', () => {
             isMobile: false,
           },
           actor: null,
+          revoke: jest.fn().mockResolvedValue({}),
         } as any as SessionWithActivitiesResource,
       ]),
     );