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

[HRIS-414] - [BE] Notifications > Index > Mark Notification as Read #326

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Miguel21Monacillo
Copy link
Contributor

Issue Link

https://framgiaph.backlog.com/view/HRIS-414

Definition of Done

  • Users can query the isRead Notification

Notes

  • The function for when the notification is read was implemented together with the index, what I did here was test it with actual data and mutation, I also added a comment on what the code does.

Pre-condition

Commands to run
-[x] npm run dev
-[x] dotnet run

Expected Output

It should be able to show the notification of the user whether admin or user.

Screenshots/Recordings

HRIS data of Admin getting a notification and viewing it
Screenshot 2024-09-10 174339

The query of notification with the same data
Screenshot 2024-09-10 174347

Copy link

@jonel371234 jonel371234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

  • Users and admin can query the isRead Notification
  • Environment: Windows 11. Google Chrome Version 130.0.6723.92 (Official Build) (64-bit)
Screen.Recording.2024-11-06.161926.mp4

Comments: 0
Test Time: 20mins
Iterations: 0

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

Successfully merging this pull request may close these issues.

2 participants