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.

Sign up for GitHub

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

Jump to bottom

Support for updating orders and synchronization of standalone products #16

Merged
merged 31 commits into from
Jul 24, 2024
Merged

Support for updating orders and synchronization of standalone products #16

merged 31 commits into from
Jul 24, 2024

Conversation

michalJakubis
Copy link
Contributor

Motivation

  • Add standalone products synchronization functionality
  • Create endpoints to update order status and retrieve order details.

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

martinfbluesoftcz and others added 26 commits May 7, 2024 17:40
@martinfbluesoftcz
Order service - update, order detail and statuses, integration tests …
1515ce4 
…for orders.
@martinfbluesoftcz
Fixed formatting
9d32531
@martinfbluesoftcz
Merge branch 'main' into feat/orders_updates
3d38642 
# Conflicts:
#	examples/DancingGoat-K13Ecommerce/Controllers/TestController.cs
#	src/Kentico.Xperience.K13Ecommerce/Orders/IOrderService.cs
#	src/Kentico.Xperience.K13Ecommerce/Orders/OrderService.cs
#	src/Kentico.Xperience.StoreApi/Orders/OrderController.cs
@martinfbluesoftcz
order payment status
51cc1cd
@BS-jiriceska
Hotfix 29.2.0
089b63b
@martinkyjac
Create endpoint to retrieve list of standalone products
2ad53db
@martinkyjac
Add get standalone products method to product service
c88ab45
@martinkyjac
Update swagger.json
71778ac
@martinkyjac
Add standalone products to product synchronization
2b6babe
@martinkyjac
Simplify query to get SKU nodes
2b3ff85
@martinkyjac
Add setting for standalone products synchronization
e93c2c0
@martinkyjac
Add SKUDescription field to KProductSKU
a207775
@martinkyjac
Add required field values
a7d6bde
@martinkyjac
Rename standalone product synchronization setting
902d519
@martinkyjac
Update README.md
485329f
@martinkyjac
Update Usage-Guide.md
d91ab3d
@martinkyjac
Fix typo
e0c0907
@martinkyjac
Merge branch 'main' into feat/orders_updates
82aaa73
@martinkyjac
Merge branch 'feat/xbyk_k13_ecommerce' into feat/orders_updates
1449d13
@martinkyjac
Add endpoint to return order info only for its customer.
43c92a7
@martinkyjac
Implement new swagger requests
01c0d65
@martinkyjac
Add SKUAvailableItems to CI dictionary
b4d7bd0
@michalJakubis
Merge pull request #12 from Kentico/feat/standalone_sku_synchronization
3e6a166 
Feat/standalone SKU synchronization
@martinkyjac
Add param description
9adc0e9
@martinkyjac
Add new endpoints to docs
64b61ee
@michalJakubis
Merge pull request #13 from Kentico/feat/orders_updates
67e2de1 
Feat/orders updates
@michalJakubis michalJakubis changed the title Support for updating orders and standalone products Support for updating orders and synchronization of standalone products Jul 19, 2024
DavidBKentico
DavidBKentico reviewed Jul 23, 2024
View reviewed changes
Copy link

@DavidBKentico DavidBKentico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kentico TW review comments added. The final review needs to be coordinated with:
#15

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/Usage-Guide.md Outdated Show resolved Hide resolved
docs/Usage-Guide.md Outdated Show resolved Hide resolved
docs/Usage-Guide.md Outdated Show resolved Hide resolved
docs/Usage-Guide.md Outdated Show resolved Hide resolved
docs/Usage-Guide.md Show resolved Hide resolved
@martinkyjac martinkyjac requested review from martinkyjac and DavidBKentico and removed request for DavidBKentico July 24, 2024 06:04
DavidBKentico
DavidBKentico approved these changes Jul 24, 2024
View reviewed changes
Copy link

@DavidBKentico DavidBKentico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, thanks for the fixes. Kentico TW review for this PR is now done.

@DavidBKentico DavidBKentico mentioned this pull request Jul 24, 2024
Merged
5 tasks
@michalJakubis michalJakubis merged commit 08b5d70 into main Jul 24, 2024
1 check passed
@michalJakubis michalJakubis deleted the feat/xbyk_k13_ecommerce branch July 24, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DavidBKentico DavidBKentico DavidBKentico approved these changes

@martinkyjac martinkyjac Awaiting requested review from martinkyjac

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@michalJakubis @DavidBKentico @martinkyjac @martinfbluesoftcz @BS-jiriceska