From 9902edcfbbdef141abbec02f08d088c2cb85fdcf Mon Sep 17 00:00:00 2001
From: Ersun Warncke <ersun.warncke@overleaf.com>
Date: Tue, 22 Oct 2019 08:29:27 -0400
Subject: [PATCH] pr fixes

GitOrigin-RevId: 15a0371c285256b871a9c3e393077af1371d1e5c
---
 services/web/app/src/Features/User/SAMLIdentityManager.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/services/web/app/src/Features/User/SAMLIdentityManager.js b/services/web/app/src/Features/User/SAMLIdentityManager.js
index af020286c7..126fc11d59 100644
--- a/services/web/app/src/Features/User/SAMLIdentityManager.js
+++ b/services/web/app/src/Features/User/SAMLIdentityManager.js
@@ -53,10 +53,6 @@ async function _addIdentifier(
     const updatedUser = User.findOneAndUpdate(query, update, {
       new: true
     }).exec()
-    // update v1 affiliations record
-    if (hasEntitlement) {
-      await InstitutionsAPI.promises.addEntitlement(userId, institutionEmail)
-    }
     return updatedUser
   } catch (err) {
     if (err.code === 11000) {
@@ -173,6 +169,10 @@ async function linkAccounts(
   )
   await _addInstitutionEmail(userId, institutionEmail, providerId)
   await _sendLinkedEmail(userId, providerName)
+  // update v1 affiliations record
+  if (hasEntitlement) {
+    await InstitutionsAPI.promises.addEntitlement(userId, institutionEmail)
+  }
 }
 
 async function unlinkAccounts(