Merge pull request #3389 from Budibase/fix/sync-issue

Quick fix for sync issue

packages/server/src/api/controllers/user.js

@@ -97,6 +97,7 @@ exports.syncUser = async function (ctx) {
       .map(([appId]) => appId)
   }
   for (let prodAppId of prodAppIds) {
+    const roleId = roles[prodAppId]
     const devAppId = getDevelopmentAppID(prodAppId)
     for (let appId of [prodAppId, devAppId]) {
       if (!(await doesDatabaseExist(appId))) {
@@ -115,19 +116,23 @@ exports.syncUser = async function (ctx) {
           tableId: InternalTables.USER_METADATA,
         }
       }
-      let combined
-      if (deleting) {
-        combined = {
+      // assign the roleId for the metadata doc
+      if (roleId) {
+        metadata.roleId = roleId
+      }
+      let combined = !deleting
+        ? combineMetadataAndUser(user, metadata)
+        : {
             ...metadata,
             status: UserStatus.INACTIVE,
             metadata: BUILTIN_ROLE_IDS.PUBLIC,
           }
-      } else {
-        combined = combineMetadataAndUser(user, metadata)
-      }
+      // if its null then there was no updates required
+      if (combined) {
         await db.put(combined)
       }
     }
+  }
   ctx.body = {
     message: "User synced.",
   }