Skip to content

Issues with Post Interview Action Reminder email notification when interview 2 is scheduled #540

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

Closed
sdgun opened this issue Sep 6, 2021 · 4 comments · Fixed by #547
Closed

Comments

@sdgun
Copy link
Collaborator

sdgun commented Sep 6, 2021

Steps
Issue 1
In the below instance, noticed two records on the same candidate interview 1 and 2. When we schedule interview 2 we shouldn't receive post interview reminder on interview 1)

image

Job title: ABC
Link to the interview: https://platform.topcoder-dev.com/taas/myteams/18690/positions/05bdcac3-b86b-4070-bfe3-483c1cb47706/candidates/interviews

Issue 2
No Post Interview Action Reminder email notification received on the below Interview 2 but received in app notification:

Interview(2):
https://platform.topcoder-dev.com/taas/myteams/18690/positions/05bdcac3-b86b-4070-bfe3-483c1cb47706/candidates/interviews

Gmail(only received interview completed email):
image

In app notifications are received as expected:
image

@sdgun sdgun changed the title Post Interview Action Reminder email notification is sent even after taking an action Issues with Post Interview Action Reminder email notification Sep 6, 2021
@sdgun sdgun changed the title Issues with Post Interview Action Reminder email notification Issues with Post Interview Action Reminder email notification when interview 2 is scheduled Sep 6, 2021
@sdgun sdgun added P2 and removed P3 labels Sep 6, 2021
@maxceem
Copy link
Contributor

maxceem commented Sep 8, 2021

Sum up:

  • For now we have to fix only the first part: avoid duplicated interviews. I guess this happens because for one JobCandidate we can re-schedule interview so there could be multiple rounds: 1, 2 and 3. So we only have to send email for the last round.
  • We also must ensure, that we don't send duplicate notification for interview completed reminder ('taas.notification.interview-awaits-resolution').
  • Please, create demo data with several rounds of interview so we could check that only 1 interview is listed in email when we run demo script https://github.com/topcoder-platform/taas-apis/tree/dev/scripts/demo-email-notifications

@maxceem
Copy link
Contributor

maxceem commented Sep 9, 2021

@maxceem maxceem added the PAID Paid manually without TCX label Sep 9, 2021
@maxceem
Copy link
Contributor

maxceem commented Sep 9, 2021

@sdgun the first part of this issue is ready for review.

QA guidelines:

  • check that if we schedule 2-3 rounds then Post Interview Action Reminder email list only the latest round for the Job Candidate
  • verify the same for Interview Awaits Resolution email

@sdgun
Copy link
Collaborator Author

sdgun commented Sep 13, 2021

Verified Post Interview Action Reminder email and Interview Awaits Resolution email with 1st, 2nd and 3rd interviews, both emails only had the latest round for the job candidate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants