Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ft schedule tech interview #282

Merged
merged 10 commits into from
Dec 9, 2024
Merged

Ft schedule tech interview #282

merged 10 commits into from
Dec 9, 2024

Conversation

bahati10
Copy link
Contributor

@bahati10 bahati10 commented Dec 2, 2024

PR Description

This PR allows the admin to invite a trainee applicant to the technical interview
by sending emails to both the applicant and interview coordinator

Description of tasks that were expected to be completed

As a trainee applicant, I will receive an email with a meeting link for the interview invitation.
As an admin, I should be able to create, view, and update the status of the interview

How can this be tested?

  • Clone repo
  • Checkout branch here
  • Make sure that you set the environment variables referring to .env.example
  • Run npm install, then npm run dev to check if the app is running successfully
  • Use Apollo server and interact with doc resolvers

Track PR (issue number & link)

Issue: #208

Screenshots (If appropriate especially on the frontend)

Screenshot 2024-12-02 at 03 14 10 Screenshot 2024-12-02 at 03 14 57 Screenshot 2024-12-02 at 03 15 55

N/A

Copy link

vercel bot commented Dec 2, 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 8, 2024 9:30am

@MUNYESHURImanzi MUNYESHURImanzi self-requested a review December 2, 2024 09:42
Copy link
Contributor

@Mugisha146 Mugisha146 left a comment

Choose a reason for hiding this comment

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

I tested this Frontend PR with your backend and find this issue:

  1. duplication on invite button
Screenshot 2024-12-03 at 08 17 03 2. On Invite you must add when to do interview means date and time Screenshot 2024-12-03 at 08 17 13 3. On your PR application stage history not displayed Screenshot 2024-12-03 at 08 19 32 4. nothing is displayed on Interview schedule on applicant side Screenshot 2024-12-03 at 08 19 45 5. check well if email is being sent because am able to see notification only not getting email ![Uploading Screenshot 2024-12-03 at 08.20.02.png…]()

@bahati10
Copy link
Contributor Author

bahati10 commented Dec 3, 2024

@Mugisha146 Thank you for mentioning these issues

  1. The issue is fixed, the duplicate has been removed
  2. The feature is from the previous merged task and is under fix too
  3. Stage history under fix too
  4. This is a different context for the interview, The only close thing an applicant can see is
    their details on my applicants page
  5. Could you provide the screenshot again, please?

@Mugisha146
Copy link
Contributor

Mugisha146 commented Dec 3, 2024

Great, all issues are resolved.

@Mugisha146 Mugisha146 closed this Dec 3, 2024
Copy link

codeclimate bot commented Dec 8, 2024

Code Climate has analyzed commit 5520adc and detected 0 issues on this pull request.

View more on Code Climate.

@Salim-54 Salim-54 merged commit eb7cb39 into develop Dec 9, 2024
7 checks passed
Salim-54 added a commit that referenced this pull request Dec 9, 2024
Salim-54 added a commit that referenced this pull request Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BYTE SWAT BYTE SWAT frontend frontend task Ready For Review Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.