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

NameX: Consent mail triggered when the condition on the NR is edited #20714

Closed
oanyahuru opened this issue Apr 5, 2024 · 2 comments
Closed
Assignees
Labels
Names Team Name Request Name Examination Team Priority1

Comments

@oanyahuru
Copy link
Collaborator

oanyahuru commented Apr 5, 2024

Examiners have noticed that for an NR that was previously approved conditionally, when they edit the NR and re-apply a condition macro;

image.png

this action incorrectly triggers a “consent received” output to the client. Please note that at this point, Consent has NOT been received, and the status on the dashboard remains "Consent Required"

image.png

This leads to obvious confusion for clients and calls to the business staff.
Reported cases so far include the following NRs:

  • NR8886487
  • NR7984245
  • NR4679401
  • NR9244243
@oanyahuru oanyahuru added the Names Team Name Request Name Examination Team label Apr 5, 2024
@semmatti semmatti self-assigned this Apr 8, 2024
@semmatti
Copy link

semmatti commented Apr 8, 2024

@oanyahuru I was able to find an alternate way to reproduce this error:

  1. Conditionally approve an NR (no need to select any conflicts, macros, conditions, etc.)
  2. Wait a couple minutes until the NR is furnished, i.e. refresh the NR page until the "Reopen" button turns into a "Reset" button.
  3. Click the "Reset" button to reset the NR
    Shortly after, an email is sent saying that a consent letter was received. This suggests that the error is related to resetting a conditionally approved NR.

@oanyahuru
Copy link
Collaborator Author

oanyahuru commented Apr 11, 2024

I tested this and waited an hour, no consent mail was received. The issue appears fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Names Team Name Request Name Examination Team Priority1
Projects
None yet
Development

No branches or pull requests

3 participants