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

bg(chat): add pagination when retrieving the messages #94

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

Conversation

jo-rdan
Copy link
Contributor

@jo-rdan jo-rdan commented Mar 26, 2020

What does this PR do?

It adds pagination functionality while retrieving messages

Description of Task to be completed?

Implement pagination functionality

How should this be manually tested?

  • Clone this repo
  • make sure you have all the packages installed by running this command npm i
  • Make sure you have all the tables in the database by running this command npm run db-migrate
  • Start the server use this endpoint /api/v1/messages?page={pass in the page number}&limit={pass in the limit number} and hit send and it should retrieve the number of records equal to the limit
    and if you change the page number it should change accordingly

Any background context you want to provide?

  • N/A

What are the relevant pivotal tracker stories?

Screenshots (if appropriate)

image

Questions:

@jo-rdan jo-rdan self-assigned this Mar 26, 2020
@jo-rdan jo-rdan force-pushed the fix/add-pagination-to-messages branch from 6c8b04f to bde151f Compare March 26, 2020 20:15
@jo-rdan jo-rdan temporarily deployed to blackninjas-backend-staging March 26, 2020 20:18 Inactive
@jo-rdan jo-rdan temporarily deployed to blackninjas-backend-staging April 1, 2020 10:06 Inactive
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

2 participants