Merge pull request #1 from Amruth-Vamshi/dev

Fixed: user registered roles being reset.
Samagra-Development · Dec 6, 2023 · 8362be1 · 8362be1
2 parents 15ad91b + 66f4127
commit 8362be1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/api/api.service.ts b/src/api/api.service.ts
@@ -564,11 +564,12 @@ export class ApiService {
           authHeader,
         );
       if (statusFA === FAStatus.USER_EXISTS) {
-        let registrationId = null;
+        let registrationId = null, registeredRoles = [];
         if (user.registrations) {
           user.registrations.map((item) => {
             if (item.applicationId == loginDto.applicationId) {
               registrationId = item.id;
+              registeredRoles = item.roles;
             }
           });
         }
@@ -581,7 +582,7 @@ export class ApiService {
             registrations: [
               {
                 applicationId: loginDto.applicationId,
-                roles: loginDto.roles ?? [],
+                roles: registeredRoles,
                 id: registrationId,
               },
             ],