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

API - Specify unit note type being cancelled on cancel note output #18150

Closed
mstanton1 opened this issue Oct 12, 2023 · 5 comments
Closed

API - Specify unit note type being cancelled on cancel note output #18150

mstanton1 opened this issue Oct 12, 2023 · 5 comments

Comments

@mstanton1
Copy link
Collaborator

mstanton1 commented Oct 12, 2023

  • during staff and policy review of unit notes it was identified that within the cancel unit note output it should be clear what type of unit note had been cancelled
  • one recommendation was to update the title 'Cancel Note' in the top right hand corner of the output to include the note type being cancelled beside that in brackets, as we do in the MHR table. However, identifying the unit note type being cancelled anywhere in the verification statement was noted as sufficient

Figma Design : https://www.figma.com/file/FVvx7KgHDCAfejklfoue3L/MHR---Unit-Notes?type=design&node-id=1792-43093&mode=design

@mstanton1
Copy link
Collaborator Author

@doug-lovett doug-lovett self-assigned this Oct 12, 2023
@doug-lovett
Copy link
Collaborator

Example NCAN document type registration report:
note-test-example.pdf

@chdivyareddy
Copy link
Collaborator

Hey @doug-lovett , I just cancelled Notice of Caution unit note and noted that the output did not display "Notice of Caution" top right hand corner of the output beside "Cancel Note" in DEV . Can you please take a look, thanks!!

MHR 107820

Private Zenhub Image

2023-10-17_BCMHR_Registration_107820.pdf

@doug-lovett
Copy link
Collaborator

@chdivyareddy I can reproduce, the change only works with the individual note registration report, not the combined staff cover letter and registration report. The fix is now in DEV.

@chdivyareddy
Copy link
Collaborator

Thank you @doug-lovett !!

Verified in DEV with MHR 107817.

image

2023-10-17_BCMHR_Registration_107817-2.pdf

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

No branches or pull requests

4 participants