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.

Sign up for GitHub

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

Jump to bottom

API controller for users with all necessary methods #234

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

API controller for users with all necessary methods #234

wants to merge 11 commits into from

Conversation

Divyeshhhh
Copy link
Collaborator

Checklist

Confirm you have completed the following actions prior to submitting this PR.

  • There is an existing issue report for this PR.
  • This is not a duplicate of an existing pull request.
  • My pull request is not resolving multiple issues at once.
  • I have created a feature branch.
  • My commit messages are detailed.
  • All of my commits are relevant to the issue.
  • composer test does not throw any error.

Issue Resolution

Tell us which issue this PR fixes.

This pull request Fixes #226

@Divyeshhhh Divyeshhhh requested a review from creme332 as a code owner June 11, 2024 17:14
@Divyeshhhh
Copy link
Collaborator Author

Divyeshhhh commented Jun 12, 2024 •
edited by creme332
Loading

  • Full setup for API Controller Users.
  • Test on Postman.
    • 'GET /api/v1/users'

image

  • 'GET /api/v1/users/[id]'
  • 'POST /api/v1/users'

image

  • 'DELETE /api/v1/users/[id]'

image

  • 'PUT /api/v1/users/[id]'
  • GET /api/v1/users/[id]/orders/

image

  • GET /api/v1/users/[id]/reviews/

image

  • Write test for Users.

I guess, I will need help for the DELETE, PUT and GET.
For the tests, once DELETE, PUT and GET is okay then i will write it.

@creme332
Copy link
Member

What errors are you getting? What did you try to fix the errors?

@Divyeshhhh
Copy link
Collaborator Author

i had edited the pull request do check it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@creme332 creme332 Awaiting requested review from creme332 creme332 is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Write API controller + tests for User endpoint
2 participants
@Divyeshhhh @creme332