Merge pull request #80 from techstartucalgary/suhaib/job_routers

Suhaib/job routers

Author: Anidion

backend/crud/user_profile_job_crud.py

@@ -194,3 +194,35 @@ def delete_applicant_application(db: Session, user_id: int, job_id: int) \
     db.delete(application)
     db.commit()
     return application
+
+def update_applicant_application_status(db: Session,
+                                        user_id: int, 
+                                        job_id: int,
+                                        new_status: application_status_model.ApplicationStatusEnum,
+                                        rejection_feedback: str | None) \
+                                        -> user_profile_job_model.UserProfileJob:
+    # Get the application
+    current_application = get_application_by_user_id_and_job_id(db, user_id, job_id)
+
+    # Get the id for the new application status
+    new_application_status_id = application_status_crud.get_application_status_by_name(db, new_status.value).id 
+
+    # Update the current applications id
+    current_application.application_status_id = new_application_status_id
+
+    # Update the other attributes of the current application depending on the new status
+    if new_status.value == application_status_model.ApplicationStatusEnum.submitted.value:
+        current_application.application_submitted_date = datetime.today()
+    elif new_status.value == application_status_model.ApplicationStatusEnum.in_review.value:
+        current_application.application_reviewed_date = datetime.today()
+    elif new_status.value == application_status_model.ApplicationStatusEnum.screening.value:
+        current_application.application_further_screening_date = datetime.today()
+    elif new_status.value == application_status_model.ApplicationStatusEnum.rejected.value:
+        current_application.application_rejected_date = datetime.today()
+        current_application.rejection_feedback = rejection_feedback
+    elif new_status.value == application_status_model.ApplicationStatusEnum.offer_sent.value:
+        current_application.application_offer_sent_date = datetime.today()
+
+    db.commit()
+    db.refresh(current_application)
+    return current_application

backend/models/user_profile_job_model.py

@@ -15,6 +15,7 @@ class UserProfileJob(Base):
     application_status_id = Column(Integer, ForeignKey("application_status.id"), nullable=False)
     application_submitted_date = Column(Date, nullable=False)
     application_reviewed_date = Column(Date, nullable=True)
+    application_further_screening_date = Column(Date, nullable=True)
     application_offer_sent_date = Column(Date, nullable=True)
     application_rejected_date = Column(Date, nullable=True)
     rejection_feedback = Column(String, nullable=True)

backend/routers/user_profile_job_router.py

@@ -1,4 +1,4 @@
-from fastapi import APIRouter, Depends, status, HTTPException, Path
+from fastapi import APIRouter, Depends, status, HTTPException, Path, Body
 
 from sqlalchemy.orm import Session
 
@@ -277,13 +277,62 @@ def delete_applicant_application(db: Session = Depends(dependencies.get_db),
     return f"Deleted application for job {job_id} for applicant {applicant.id}."
 
 
-# PATCH /applications/{job_id}_{applicant_id}/review change status to under review
-# @router.patch(
-#     "/applications/review/"
-# )
+# PATCH /applications/{job_id}_{applicant_id}/ change status 
+@router.patch(
+    "/applications/{job_id}_{applicant_id}/",
+    response_model=user_profile_job_schema.UserProfileJob,
+    status_code=status.HTTP_200_OK,
+    tags=["Application"],
+    summary="PATCH route for a recruiter to change the status of an application.",
+    description="Change the status of an application for a job and provide optional rejection feedback " \
+        "for the applicant. The status of an application cannot move backwards. The recruiter must " \
+        "own the job to be able to change the status of an application.",
+    response_description="The updated application."
+)
+def change_application_status(db: Session = Depends(dependencies.get_db),
+                               job_id: int = Path(),
+                               applicant_id: int = Path(),
+                               new_status: application_status_model.ApplicationStatusEnum = Body(),
+                               rejection_feedback: str | None = Body(default=None),
+                               recruiter: user_model.User = Depends(dependencies.get_current_recruiter_user)):
+    # Check the application exists
+    application = user_profile_job_crud.get_application_by_user_id_and_job_id(db, applicant_id, job_id)
+    if application is None:
+        raise HTTPException(
+            status_code=status.HTTP_404_NOT_FOUND,
+            detail=f"Application for job {job_id} and applicant {applicant_id} does not exist."
+        )
+
+    # Check the recruiter owns the job
+    job = job_crud.get_job_by_id(db, job_id)
+
+    if job.user_profile_id != recruiter.id:
+        raise HTTPException(
+            status_code=status.HTTP_401_UNAUTHORIZED,
+            detail=f"Recruiter {recruiter.id} is not authorized to update applications for job " \
+                "{job_id}."
+        )
+    
+    # Check the current status of the application and the new status of the application to see if it is allowed
+    # The new status of an application must be greater or equal to the current status of an application
+    # except in the case of REJECTED, which is a final status it can only be equal
+    can_change_app_status = False
+    new_application_status_id = application_status_crud.get_application_status_by_name(db, new_status.value).id
+
+    if application.application_status.status == application_status_model.ApplicationStatusEnum.rejected.value:
+        if new_application_status_id == application.application_status_id:
+            can_change_app_status = True
+    else:
+        if new_application_status_id >= application. application_status_id:
+            can_change_app_status = True
 
-# PATCH /applications/{job_id}_{applicant_id}/further_screening 
+    if not can_change_app_status:
+        raise HTTPException(
+            status_code=status.HTTP_400_BAD_REQUEST,
+            detail=f"Cannot change application status from {application.application_status.status} to "\
+                f"{new_status.value}."
+        )
 
-# PATCH /applications/{job_id}_{applicant_id}/offer change status to offer sent
+    # Update the application status of the application
+    return user_profile_job_crud.update_applicant_application_status(db, applicant_id, job_id, new_status, rejection_feedback)
 
-# PATCH /applications/{job_id}_{applicant_id}/rejected change status to rejected

backend/schemas/user_profile_job_schema.py

@@ -44,6 +44,7 @@ class UserProfileJob(UserProfileJobBase):
     application_status_id : int
     application_submitted_date : date | None
     application_reviewed_date : date | None
+    application_further_screening_date : date | None
     application_offer_sent_date : date | None
     application_rejected_date : date | None
     rejection_feedback : str | None
@@ -59,12 +60,13 @@ class Config:
                 "job_id": 1,
                 "application_status_id": 1,
                 "application_submitted_date": "2023-01-01",
-                "application_reviewed_date": None,
-                "application_offer_sent_date": None,
+                "application_reviewed_date": "2023-01-02",
+                "application_further_screening_date": "2023-01-02",
+                "application_offer_sent_date": "2023-01-03",
                 "application_rejected_date": None,
                 "application_status": {
                     "id": 1,
-                    "status": "Application Submitted"
+                    "status": "SUBMITTED"
                 },
                 "applicant": {
                     "user_id": 1,

backend/sql/cyberhire_ddl.sql

@@ -84,10 +84,11 @@ CREATE TABLE IF NOT EXISTS cyberhire."application_status" (
 
 INSERT INTO cyberhire."application_status" (status)
 VALUES 
-('Application Submitted'),
-('Application Under Review'),
-('Offer Sent'),
-('Rejected')
+('SUBMITTED'),
+('UNDER REVIEW'),
+('UNDERGOING FURTHER SCREENING')
+('REJECTED'),
+('OFFER SENT')
 ;
 
 -- Create for the user_profile_job table
@@ -97,11 +98,12 @@ CREATE TABLE IF NOT EXISTS cyberhire."user_profile_job" (
 	application_status_id int4 NOT NULL,
 	application_submitted_date date NOT NULL,
 	application_reviewed_date date NULL,
+	application_further_screening_date date NULL,
 	application_offer_sent_date date NULL,
 	application_rejected_date date NULL,
 	rejection_feedback varchar(2000) NULL,
 	CONSTRAINT user_profile_job_pkey PRIMARY KEY (user_profile_id, job_id),
 	CONSTRAINT user_profile_job_fk FOREIGN KEY (user_profile_id) REFERENCES cyberhire."user_profile" (user_id),
 	CONSTRAINT user_profile_job_fk2 FOREIGN KEY (job_id) REFERENCES cyberhire."job" (id),
 	CONSTRAINT user_profile_job_fk3 FOREIGN KEY (application_status_id) REFERENCES application_status (id)
-);
+);