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

API controller for orders with all necessary methods #235

Open
wants to merge 3 commits into
base: main
Choose a base branch
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 #225

@Divyeshhhh Divyeshhhh requested a review from creme332 as a code owner June 12, 2024 06:56
@Divyeshhhh
Copy link
Collaborator Author

Divyeshhhh commented Jun 12, 2024

  • Full setup for API Controller Orders.
  • Test on Postman.
    • 'GET /api/v1/orders'
    • 'GET /api/v1/orders/[id]'
    • 'POST /api/v1/orders'
    • 'PUT /api/v1/orders/[id]'
    • 'DELETE /api/v1/orders/[id]'
  • Write test for Orders.

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

@Divyeshhhh
Copy link
Collaborator Author

do you solve it?

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.

Write API controller + tests for Order endpoint
2 participants