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 technical interview #169

Merged
merged 16 commits into from
Dec 9, 2024
Merged

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

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)

N/A

@MUNYESHURImanzi MUNYESHURImanzi self-requested a review December 2, 2024 09:45
* fix: remove the rm step

* test: run on PR

* test: test pusher cluster env

* fix: add notification envs

* test: add pusher key

* fix: add the pusher app key

* fix: log container start output

* fix: remove the e flag

* fix: revert changes

* fix: re-add the pusher app key

* fix: add pusher app key value

* fix: typo

* fix: test env file

* fix: syntax

* fix: remove the env from run cmd

* fix: revert changes

* fix: revert changes

* fix: wrap run cmd in an if statement

* Revert "test: test pusher cluster env"

This reverts commit 57b2bbe.

* test: create env file

* test: add error handling
- remove 'submitted' status
- add comments for 'rejected status'
ceelogre and others added 6 commits December 6, 2024 13:19
* fix: don't exit on cmd failure

* fix: re-add values

* fix: add new variables

* test: add container name

* fix: match ports

* fix: add devpulse email

* fix: port

* fix: port

* fix: remove container name

* fix: remove extraneous node env

* fix: add jwt key

* fix: run on push
* fix (170): an author will be able to update a blog

* fix (170): allow all users to create and update blogs
@render render bot temporarily deployed to ft-schedule-technical-interview - dev-pulse-bn PR #169 December 6, 2024 12:02 Destroyed
@ceelogre ceelogre temporarily deployed to ft-schedule-technical-interview - atlp-devpulse-bn PR #169 December 6, 2024 12:02 — with Render Destroyed
@Salim-54 Salim-54 merged commit eb83c2b into develop Dec 9, 2024
1 check 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants