Open ozamani9gh opened 4 months ago
@ozamani9gh - This requires access to gc notify which we only have a possible servie to hit and may need some work from SRE.
as a first step, based on the requirements, may need to create a record of email contents in the namex-db at the time message is fired off to notify api: https://github.com/bcgov/namex/blob/b9a437b0c5ff0feffd3333ba83d1f05cc5e01651/api/namex/resources/name_requests/report_resource.py#L144 the resend button should result in calling that method as well
is there is requirement to also pull notification states from notifi-api? that might be - an addition after the first step is implemented
This is brand new functionality ... alot of work to visualize email history. can we see if its worth the effort
@oanyahuru @Mihai-QuickSilverDev this will be several sprints worth of effort. Can we confirm this is very important for Names Examiners to have a history of email notifications. if it is, i will need to probably convert this to epic and create smaller story points
@oanyahuru Hi Oge, please let me know if you want me to follow-up with the Name Examiners on this, or if you would prefer to do it. Thanks much!
@Mihai-QuickSilverDev I will follow up
@ozamani9gh I discussed with the business and they again confirmed this is important. Having the notification history accessible at the click of a button is very useful for a name examiner assisting a client live on the phone for instance.
The UI for this is finished. However the API needs an endpoint to be able to fetch notification (previous email) history for a certain NR number.
Endpoint Should be at this location
GET .../api/v1/notifications/
Endpoint should return this data
@ozamani9gh
Putting this ticket on hold. Waiting for API to be implemented.
Waiting on NAMEX API to move to GCP
Notification History Button and Navigation to Notification History Page (could be opened in new tab) and Navigate Back functionality.
https://www.figma.com/file/zHZFygfAWSmNp6kDb7g1Zs/NameX-Design-Tickets?type=design&node-id=2-25&mode=design