Skip to content

Conversation

@SaddockAime
Copy link
Contributor

@SaddockAime SaddockAime commented Dec 3, 2024

PR Description

this PR makes sure that we are only adding admitted trainee in cohort.

Related Issue

#290

Changes Made

  • made sure that we are only adding admitted trainee in the cohort

How to Test This PR:

Locally:

  • Clone the repository.
  • Run npm install to install dependencies and navigate to the appropriate the branch fix-add-trainee-in-cohort
  • Start the server using npm run dev
  • first login as applicant with email: [email protected], password: password123 to apply for becoming a trainee
  • then login as admin with email: [email protected], password: password123 and check in the application cycle to admit the applicant who requested to become trainee.
  • after admitting the applicant to became a trainee, go to cohort and choose one cohort you want to add that trainee into and then view it and add that trainee in that cohort.

Deployed:

  • first login as applicant with email: [email protected], password: password123 to apply for becoming a trainee
  • then login as admin with email: [email protected], password: password123 and check in the application cycle to admit the applicant who requested to become trainee.
  • after admitting the applicant to became a trainee, go to cohort and choose one cohort you want to add that trainee into and then view it and add that trainee in that cohort.

Screenshots:

image
image
image

@vercel
Copy link

vercel bot commented Dec 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2024 4:42pm

Copy link
Contributor

@ncutixavier ncutixavier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove all the logs

@SaddockAime SaddockAime force-pushed the fix-add-trainee-in-cohort branch from 97f9fd2 to 6bb501f Compare December 4, 2024 16:39
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 6bb501f and detected 0 issues on this pull request.

View more on Code Climate.

@ncutixavier ncutixavier merged commit 066fd7d into develop Dec 5, 2024
7 checks passed
@ncutixavier ncutixavier deleted the fix-add-trainee-in-cohort branch December 5, 2024 08:03
bahati10 pushed a commit that referenced this pull request Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DREAM DREAM TEAM Ready For Review Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add trainee in cohort

7 participants