Bookmark Icon Not Functioning Correctly for Important Notifications

[Jeffrin2005]

Describe the bug

Starred icon is visible but non-functional.
When clicking the bookmark icon to tag an important notification, it redirects to the "Log Update" section instead of marking the notification.

To Reproduce

Steps to reproduce the behavior:

1. Go to 'Notification Tab'
2. Click on the bookmark icon
3. Observe that it does not mark the notification as important and navigates to another page instead

Expected behavior

The bookmark icon should be marked in red when a notification is tagged as important.

Record Clip

Bookmarking.mp4

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

[Jeffrin2005]

@rithviknishad @Jacobjeevan I would like to work on this. Could you please assign it to me?

[Jacobjeevan]

That's not a bookmark icon, but rather patient log update icon (speaking of which, we might have to update the icon there 🤔 ).

As such, clicking on it should take to patient log update page - consultation/consultation:id/log_updates/log:id/critical_care. You can navigate it to by visiting a patient consultation page (click on any patient with consultation) and checking the log updates tab for events (and click on view details).

[Jeffrin2005]

@Jacobjeevan

Changes Made

• Once the User clicks the Bookmark icon it turned out to Red
• For persisting the State, used local Storage.

bookmark.mp4

[Jeffrin2005]

@Jacobjeevan Please let me know if any further changes are required before PR.

[Jacobjeevan]

@Jacobjeevan

Changes Made

* Once the User clicks the Bookmark icon it turned out to Red

* For persisting the State, used local Storage.

👇

That's not a bookmark icon, but rather patient log update icon (speaking of which, we might have to update the icon there 🤔 ).

As such, clicking on it should take to patient log update page - consultation/consultation:id/log_updates/log:id/critical_care. You can navigate it to by visiting a patient consultation page (click on any patient with consultation) and checking the log updates tab for events (and click on view details).

[Jeffrin2005]

@Jacobjeevan
Sorry for the misunderstanding. For confirmation, after clicking the "log update" icon, it should navigate to this page, right? But when we click the log update icon for each person in notifications tab, how do we know which person's details it corresponds to?

[Jacobjeevan]

@Jacobjeevan Sorry for the misunderstanding. For confirmation, after clicking the "log update" icon, it should navigate to this page, right? But when we click the log update icon for each person in notifications tab, how do we know which person's details it corresponds to?

No, it should navigate to the log updates detail page; clicking on the view details button (check the screenshot I attached above) takes you to the relevant page.

I looked into it, and it's a BE issue; the URL is technically accurate, however the log update Id stored and retrieved from BE is incorrect. I went ahead and opened a Backend PR to address it.