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

#166815539 fix bug user read unread notification #103

Merged
merged 2 commits into from
Jun 20, 2019

Conversation

Inclet
Copy link
Contributor

@Inclet Inclet commented Jun 20, 2019

What does this PR do?

This PR retrieve notification of a user and sort them by unread ones.

Description of Task to be completed?

  • Included an attribute of sorting notifications as an instance method of notification model.

How should this be manually tested?

  • Clone this repo https://github.com/andela/strikers-ah-backend.git
  • Open a folder where you cloned this repo and open Terminal/cmd there.
  • Checkout to develop branch by typing git checkout develop in your opened terminal.
  • Type npm install to install all dependencies needed for this app.
  • Type npm start to start the server.
  • To see all notification you can access localhost:5000/api/profiles/notifications route through postman.

Any background context you want to provide?

  • Use postman chrome extension to access app routes.

What are the relevant pivotal tracker stories?

#166815539

Screenshots (if appropriate)

Screen Shot 2019-06-20 at 16 00 12

@Inclet Inclet temporarily deployed to strikers-ah-backend-sta-pr-103 June 20, 2019 14:01 Inactive
@Inclet Inclet requested review from mwibutsa and jnkindi June 20, 2019 14:01
Copy link
Contributor

@mwibutsa mwibutsa left a comment

Choose a reason for hiding this comment

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

Fix the Travis issues and let us know

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

Successfully merging this pull request may close these issues.

None yet

2 participants