fix: set the right status for adding and deleting memebers

packages/insea-connect-backend/src/main/java/ma/insea/connect/chat/group/GroupController.java

@@ -8,6 +8,7 @@
 import java.util.List;
 import java.util.Map;
 
+import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
@@ -54,13 +55,22 @@ public ResponseEntity<List<UserDTO3>> findGroupUsers(@PathVariable("groupId") Lo
 
     @PostMapping("/groups/{groupId}/members")
     public ResponseEntity<String> addGroupMembers(@PathVariable("groupId") Long groupId, @RequestBody Map<String, List<Long>>users) {
+        Boolean added=groupService.addGroupMembers(groupId, users.get("members"));
+        if(!added){
+            return new ResponseEntity(HttpStatus.UNAUTHORIZED);}
+        else{
+            return new ResponseEntity(HttpStatus.OK);}
+
 
-        return ResponseEntity.ok(groupService.addGroupMembers(groupId, users.get("members")));
     }
 
     @DeleteMapping("/groups/{groupId}/members/{memberid}")
     public ResponseEntity<String> removeGroupMember(@PathVariable("groupId") Long groupId, @PathVariable("memberid") Long memberId) {
-        return ResponseEntity.ok(groupService.removeGroupMember(groupId, memberId));
+        Boolean deleted=groupService.removeGroupMember(groupId, memberId);
+        if(!deleted){
+            return new ResponseEntity(HttpStatus.UNAUTHORIZED);}
+        else{
+            return new ResponseEntity(HttpStatus.OK);}
     }
     @PostMapping("/groups/{groupId}/admins")
     public ResponseEntity<?> addAdmin(@RequestBody Map<String, Long> userId, @PathVariable("groupId") Long groupId) {

packages/insea-connect-backend/src/main/java/ma/insea/connect/chat/group/GroupService.java

@@ -111,12 +111,12 @@ public List<UserDTO3> findUsers(Long groupId) {
         return groupMembersDTOs;
 
     }
-    public String addGroupMembers(Long groupId, List<Long> users) {
+    public Boolean addGroupMembers(Long groupId, List<Long> users) {
         User connectedUser = functions.getConnectedUser();
         Membership membership = membershipRepository.findByUserIdAndGroupId(connectedUser.getId(), groupId);
 
         if(membership == null || !membership.getIsAdmin()) {
-            return "You are not allowed to add members to this group";
+            return false;
         }else{
         for (Long user : users) {
             Membership m = new Membership();
@@ -127,20 +127,20 @@ public String addGroupMembers(Long groupId, List<Long> users) {
             m.setJoiningDate(new java.util.Date(System.currentTimeMillis()));
             membershipRepository.save(m);
         }
-        return "Group members added successfully";
+        return true;
     }
 
     }
     @Transactional
-    public String removeGroupMember(Long groupId, Long memberId) {
+    public Boolean removeGroupMember(Long groupId, Long memberId) {
         User connectedUser = functions.getConnectedUser();
         Membership membership = membershipRepository.findByUserIdAndGroupId(connectedUser.getId(), groupId);
 
         if(membership == null || !membership.getIsAdmin()) {
-            return "You are not allowed to add members to this group";
+            return false;
         }else{
         membershipRepository.deleteByGroupIdAndUserId(groupId, memberId);
-        return "Group member removed successfully";}
+        return true;}
     }
     public GroupDTO3 getGroup(Long groupId) {
         Group group = groupRepository.findById(groupId).get();