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

Backend Orders API #12

Open
wants to merge 3 commits into
base: setup-backend
Choose a base branch
from
Open

Backend Orders API #12

wants to merge 3 commits into from

Conversation

AsumVictor
Copy link
Collaborator

Backend Orders API

In this PR, I have added order routes for CRUD operation and simple information.

Type of change

  • Created a new folder called controllers to store files for CRUD logic
  • Updated the order file in the model directory
  • Updated the order file in the routes
  • Add a function to delete all completed todos in the list.

Additional changes that weren't in the project requirement

Pickup station routes

The pickup station is where customers will go and pick up or take their items they ordered. It would be advisable to create a table for all of the available pick-up stations. Customers will select a pickup station at the time of placing an order then we link the selected pick station to the delivery address.

Why

  • These will provide an efficient way for users to choose the exact address instead of typing it manually
  • It would also help to get all orders from a particular pick station for proper analysis

CHANGES

  • Created a route file for the pickup station
  • created a controller file for the pick-up station for CRUD operations
  • Created a model for the pickup station

@netlify
Copy link

netlify bot commented Jun 22, 2023

Deploy Preview for patupa failed.

Name Link
🔨 Latest commit a9ee64e
🔍 Latest deploy log https://app.netlify.com/sites/patupa/deploys/6494b71540d4f000086f25b1

@AsumVictor AsumVictor changed the base branch from main to setup-backend June 22, 2023 21:16
@AsumVictor
Copy link
Collaborator Author

Fix

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.

1 participant