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

Additional endpoint for get order and by group #107

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

jamalsoueidan
Copy link
Owner

No description provided.

…s for

getting bookings by group and booking range
feat(customer-order): remove redundant controllers for getting orders by group
and order range
feat(customer-booking): add aggregation pipeline for customer booking data

feat(booking): add functionality to retrieve orders by group
feat(booking): add functionality to retrieve orders within a specified range

refactor(get-by-group.ts): remove unused imports and variables
feat(get.spec.ts): add creation of shipping entity for testing purposes

refactor(get.ts): update CustomerOrderServiceGetAggregate type definition
feat(get.ts): add CustomLineItems type for more specific line item structure
docs(get.ts): add comment explaining purpose of CustomerOrderServiceGet method

chore(range.ts): remove unused CustomerOrderServiceRange function and types
feat(openapi): add client schema for orders
feat(openapi): add customer schema for orders
feat(openapi): add fulfillment schema for orders
feat(openapi): add line item schema for orders

feat(openapi): add new YAML files for order-related types and properties

refactor(order.yaml): refactor order schema to use base-order schema and add line_items property
fix(response.yaml): update payload reference to use correct order schema
@jamalsoueidan jamalsoueidan merged commit 6e43814 into main Mar 1, 2024
2 checks passed
@jamalsoueidan jamalsoueidan deleted the additional-endpoint-for-get-order-and-by-group branch March 1, 2024 14:27
@jamalsoueidan jamalsoueidan self-assigned this Mar 14, 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.

1 participant