From f77894c427ff230f0e6b27bec98932b513094116 Mon Sep 17 00:00:00 2001 From: Jimmy Domagala-Tang Date: Tue, 2 Jul 2024 09:31:12 -0400 Subject: [PATCH] Merge pull request #19238 from overleaf/jdt-use-ai-project-fix feat: matching rename to checkPermissions within projectController GitOrigin-RevId: ea05800c145858237fb0b236f9a19837c6c4a9a2 --- .../web/app/src/Features/Authorization/PermissionsManager.js | 2 ++ services/web/app/src/Features/Project/ProjectController.js | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/services/web/app/src/Features/Authorization/PermissionsManager.js b/services/web/app/src/Features/Authorization/PermissionsManager.js index fca40443d4..aafcf9f17b 100644 --- a/services/web/app/src/Features/Authorization/PermissionsManager.js +++ b/services/web/app/src/Features/Authorization/PermissionsManager.js @@ -450,9 +450,11 @@ module.exports = { getUserRestrictions, getUserValidationStatus: callbackify(getUserValidationStatus), checkCollaboratorsPermission: callbackify(checkUserListPermissions), + checkUserPermissions: callbackify(checkUserPermissions), promises: { assertUserPermissions, getUserValidationStatus, checkUserListPermissions, + checkUserPermissions, }, } diff --git a/services/web/app/src/Features/Project/ProjectController.js b/services/web/app/src/Features/Project/ProjectController.js index 191033e3bd..e37e1a9eac 100644 --- a/services/web/app/src/Features/Project/ProjectController.js +++ b/services/web/app/src/Features/Project/ProjectController.js @@ -613,10 +613,10 @@ const _ProjectController = { !showPersonalAccessToken && splitTestAssignments['personal-access-token'].variant === 'enabled' // `?personal-access-token=enabled` + // still allow users to access project if we cant get their permissions, but disable AI feature let canUseAi try { - await checkUserPermissions(user, ['use-ai']) - canUseAi = true + canUseAi = await checkUserPermissions(user, ['use-ai']) } catch (err) { canUseAi = false }