Merge pull request #513 from chaynHQ/develop

Merge Develop onto Main

.github/pull_request_template.md

@@ -4,6 +4,14 @@
 
 ### Why did you make the changes?
 
+### Did you run tests?
+
+<!---IMPORTANT NOTE ABOUT RUNNING TESTS: ->
+- Some tests may require multiple runs before success.
+- Some test failures may not be due to your contribution and can be ignored.
+- If tests fail without resolution, please let us know here.
+- Directions for running tests are in the README.md 
+
 <!--- PR CHECKLIST: PLEASE REMOVE BEFORE SUBMITTING —>
 Before submitting, check that you have completed the following tasks:
 - [ ] Answered the questions above.

src/partner-admin/partner-admin.service.ts

@@ -83,11 +83,18 @@ export class PartnerAdminService {
     if (!partnerAdminResponse) {
       throw new HttpException('Partner admin does not exist', HttpStatus.BAD_REQUEST);
     }
-    return this.partnerAdminRepository
+    const updatedPartnerAdminResponse = await this.partnerAdminRepository
       .createQueryBuilder('partner_admin')
       .update(PartnerAdminEntity)
       .set({ active: updatePartnerAdminDto.active })
       .where('partnerAdminId = :partnerAdminId', { partnerAdminId })
-      .returning('*');
+      .returning('*')
+      .execute();
+
+    if (updatedPartnerAdminResponse.raw.length > 0) {
+      return updatedPartnerAdminResponse.raw[0];
+    } else {
+      throw new Error('Failed to update partner admin');
+    }
   }
 }