Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Models and Schemas, Batch 2: Customers, Products, and Orders #130

Merged
merged 6 commits into from
Jul 26, 2024

Conversation

slaarti
Copy link
Contributor

@slaarti slaarti commented Jul 17, 2024

Description

This is the second batch of models and schemas for #72, covering Customer, Product, and Order.

Changes

Models, schemas, and doc/docstring updates for Customer, Product, and Order.

Tests

No changes to tests; this is purely in documentation and examples.

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • I have added necessary documentation (if appropriate).

@slaarti
Copy link
Contributor Author

slaarti commented Jul 22, 2024

Reworked the commits some to focus the PR on just this set of models/schemas and docs related to same, in an effort to better avoid stepping on my own toes across PRs.

@slaarti slaarti force-pushed the models_custprodord branch from a74cd0d to d567baf Compare July 25, 2024 15:58
@igorbenav
Copy link
Owner

Again, great pr. Thanks!

@igorbenav igorbenav merged commit 7705086 into igorbenav:main Jul 26, 2024
7 checks passed
@slaarti slaarti deleted the models_custprodord branch July 26, 2024 14:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants