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

#165413117 Get list of users #34

Merged
merged 1 commit into from
May 21, 2019
Merged

Conversation

AJAkimana
Copy link
Contributor

@AJAkimana AJAkimana commented May 20, 2019

Description

Using Author Heaven, as a user, you will need to follow or unfollow the authors. This PR is about listing users and shows the authors that user is following and the ones which are followers.

Type of change

N/A

How Has This Been Tested?

This can be tested by visiting the following route


- Get a list of all users:   GET  /api/v1/users

Checklist:

N/A

Pivotal Tracker ID

165413117

@AJAkimana AJAkimana added the peer review This PR is ready for review label May 20, 2019
@AJAkimana AJAkimana changed the title Get list of users #165413117 Get list of users May 20, 2019
@Niyitangasam
Copy link
Contributor

You can correct your PR description, you are using GET not POST as you mentioned.

@jabichris
Copy link
Contributor

@AJAkimana Get a list of all users: POST /api/v1/users . As Samuel mentioned before, are you Getting or Posting?

@AJAkimana AJAkimana force-pushed the ft-get-users-list-165413117 branch from 87492d8 to 02bc1ef Compare May 21, 2019 07:22
- Set up route and controller
- List users
- Write API documentation and test

[Finishes #165413117]
@AJAkimana AJAkimana force-pushed the ft-get-users-list-165413117 branch from 02bc1ef to 21408f3 Compare May 21, 2019 12:27
@denislohan denislohan merged commit d7d0cf6 into develop May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
peer review This PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants