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

fix: add order table and seeder #388

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Conversation

timiajayi
Copy link
Contributor

Description

Added order table and seeder to the project. This includes creating a migration for the orders table, implementing an Order model, and creating a seeder to populate the table with sample data.

Related Issue (Link to Github issue)

Motivation and Context

The addition of the order table and seeder is required to support order management functionality in the application. This change provides the necessary database structure and sample data for orders.

How Has This Been Tested?

The changes have been tested by running the migration and seeder. The order table was successfully created in the database, and sample order data was inserted without errors.

Screenshots (if appropriate - Postman, etc):

Screenshot (127)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@Muhammad235 Muhammad235 self-requested a review August 6, 2024 19:04
@Dev-Tonia Dev-Tonia merged commit 6be653d into hngprojects:dev Aug 6, 2024
1 check passed
@timiajayi timiajayi self-assigned this Aug 18, 2024
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.

3 participants