feat: Add API endpoint for active users

- Add UsersActiveApi client class
- Update users API client to include active endpoint
- Implement AllActiveUsersAPIView for active users
- Add URL path for active users endpoint

Author: Mahmoud-Emad

client/src/clients/api/users.ts

@@ -11,6 +11,7 @@ export class UsersApi extends ApiClientBase {
   readonly skills: UsersSkillsApi
   readonly team: UsersTeamApi
   readonly supervisors: UsersSupervisorsApi
+  readonly active: UsersActiveApi
 
   constructor(options: Api.ClientOptions) {
     super(options)
@@ -22,6 +23,7 @@ export class UsersApi extends ApiClientBase {
     this.skills = new UsersSkillsApi(options, this.path)
     this.team = new UsersTeamApi(options, this.path)
     this.supervisors = new UsersSupervisorsApi(options, this.path)
+    this.active = new UsersActiveApi(options, this.path)
   }
 
   list(query?: any) {
@@ -155,3 +157,11 @@ class UsersTeamSupervisorsApi extends ApiClientBase {
     )
   }
 }
+
+class UsersActiveApi extends ApiClientBase {
+  protected readonly path = '/active'
+
+  async list() {
+    return this.unwrap(() => this.$http.get(this.getUrl()))
+  }
+}

client/src/views/RequestsView.vue

@@ -210,11 +210,10 @@ export default defineComponent({
     const fetchUsers = async () => {
       isLoadingUsers.value = true;
       try {
-        // Use general list endpoint which is accessible by all authenticated users
-        const result = await $api.users.list({ page_size: 1000 });
-        if (result) {
-          users.value = result.results;
-        }
+        // Use the new accessible active users endpoint
+        // This returns directly the array of users (data)
+        const result: any = await $api.users.active.list();
+        users.value = result;
       } catch (e) {
         console.error("Failed to fetch users", e)
       } finally {

server/cshr/routes/users.py

@@ -17,11 +17,13 @@
     SupervisorsAPIView,
     PostUserSkillsAPIView,
     GetUsersBirthDatesAPIView,
+    AllActiveUsersAPIView,
 )
 
 
 urlpatterns = [
     path("", BaseGeneralUserAPIView.as_view()),
+    path("active/", AllActiveUsersAPIView.as_view()),
     path("set_inactive/", SetInActiveUserAPIView.as_view()),
     path("set_active/", SetActiveUserAPIView.as_view()),
     path("skills/", UserSkillsAPIView.as_view()),

server/cshr/views/users.py

@@ -60,11 +60,6 @@ def get_queryset(self) -> Response:
             query_set = get_all_of_users(options)
         else:
             query_set = get_all_of_users()
-
-        full_name = self.request.query_params.get("full_name")
-        if full_name:
-             query_set = query_set.filter(first_name__icontains=full_name)
-
         return query_set
 
 
@@ -425,3 +420,28 @@ def get(self, request: Request) -> Response:
         query_set = get_admins_and_supervisors()
         serializer = self.get_serializer(query_set, many=True)
         return CustomResponse.success(data=serializer.data)
+
+
+class AllActiveUsersAPIView(GenericAPIView):
+    """
+    A view for retrieving a list of all active users without pagination.
+    Accessible to all authenticated users.
+    """
+
+    serializer_class = GeneralUserSerializer
+    permission_classes = [UserIsAuthenticated]
+
+    def get(self, request: Request) -> Response:
+        # Get all users (which includes active filtering if we use the service usually or we filter explicitly)
+        # get_all_of_users returns queryset ordered by name.
+        # We need to ensure we only get active uses.
+        
+        from cshr.services.users import get_all_of_users
+        
+        # get_all_of_users() implementation:
+        # queryset = User.objects.filter(is_active=True)
+        # ... and handles options.
+        
+        query_set = get_all_of_users()
+        serializer = self.get_serializer(query_set, many=True)
+        return CustomResponse.success(data=serializer.data)