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

#167891594: User Marks Notifications as read #68

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

Conversation

tvpeter
Copy link
Collaborator

@tvpeter tvpeter commented Sep 17, 2019

What does this PR do?

  • Creates an endpoint for users to mark all in-app notifications as read

Description of Task to be completed?

  • Check that is either travel admin or manager that is logged in
  • Select a users notifications and update their status

How should this be manually tested?

  • on postman, send a patch request with only token to api/v1/notifications/read to receive a response.

Any background context you want to provide?

-none

What are the relevant pivotal tracker stories?

#167891594

Screenshots (if appropriate)

Screen Shot 2019-09-17 at 5 19 22 AM

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.

None yet

1 participant