Parcourir la source

Merge pull request #6386 from overleaf/em-promisify

Promisify UserAdminController

GitOrigin-RevId: 51bf4ddd0673409006658fb0d8cc60a8b4e1071d
Eric Mc Sween il y a 4 ans
Parent
commit
fd60549482

+ 2 - 1
services/web/app/src/Features/Institutions/InstitutionsGetter.js

@@ -1,4 +1,4 @@
-const { callbackify } = require('util')
+const { promisify, callbackify } = require('util')
 const UserGetter = require('../User/UserGetter')
 const UserMembershipsHandler = require('../UserMembership/UserMembershipsHandler')
 const UserMembershipEntityConfigs = require('../UserMembership/UserMembershipEntityConfigs')
@@ -80,6 +80,7 @@ const InstitutionsGetter = {
 InstitutionsGetter.promises = {
   getCurrentInstitutionIds,
   getCurrentInstitutionsWithLicence,
+  getManagedInstitutions: promisify(InstitutionsGetter.getManagedInstitutions),
 }
 
 module.exports = InstitutionsGetter

+ 3 - 0
services/web/app/src/Features/Publishers/PublishersGetter.js

@@ -15,6 +15,7 @@
 let PublishersGetter
 const UserMembershipsHandler = require('../UserMembership/UserMembershipsHandler')
 const UserMembershipEntityConfigs = require('../UserMembership/UserMembershipEntityConfigs')
+const { promisifyAll } = require('../../util/promises')
 const logger = require('@overleaf/logger')
 const _ = require('underscore')
 
@@ -30,3 +31,5 @@ module.exports = PublishersGetter = {
     )
   },
 }
+
+module.exports.promises = promisifyAll(PublishersGetter)

+ 3 - 0
services/web/app/src/Features/Spelling/SpellingHandler.js

@@ -1,5 +1,6 @@
 const OError = require('@overleaf/o-error')
 const Metrics = require('@overleaf/metrics')
+const { promisifyAll } = require('../../util/promises')
 const LearnedWordsManager = require('./LearnedWordsManager')
 
 module.exports = {
@@ -24,3 +25,5 @@ module.exports = {
     LearnedWordsManager.deleteUsersLearnedWords(userId, callback)
   },
 }
+
+module.exports.promises = promisifyAll(module.exports)

+ 1 - 0
services/web/app/src/Features/User/UserUpdater.js

@@ -399,6 +399,7 @@ const UserUpdater = {
 const promises = {
   addAffiliationForNewUser: promisify(UserUpdater.addAffiliationForNewUser),
   addEmailAddress,
+  clearSAMLData,
   confirmEmail,
   setDefaultEmailAddress,
   updateUser: promisify(UserUpdater.updateUser),