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

Add onHand to product response #503

Merged
merged 1 commit into from
Jul 31, 2019

Conversation

alexander-schranz
Copy link
Contributor

It should be possible to know how many products are still available.

@alexander-schranz alexander-schranz requested a review from a team as a code owner July 30, 2019 19:29
@alexander-schranz alexander-schranz force-pushed the feature/product-on-hand branch 8 times, most recently from 310dadf to 8a91261 Compare July 30, 2019 20:07
@mamazu
Copy link
Member

mamazu commented Jul 30, 2019

Could you rebase this onto the current master?

@alexander-schranz alexander-schranz force-pushed the feature/product-on-hand branch from 8a91261 to 168ca19 Compare July 31, 2019 07:44
@alexander-schranz
Copy link
Contributor Author

@mamazu all rebased

@mamazu mamazu merged commit d4ebf04 into Sylius:master Jul 31, 2019
@mamazu
Copy link
Member

mamazu commented Jul 31, 2019

Thanks, Alexander! 🎉

@alexander-schranz alexander-schranz deleted the feature/product-on-hand branch July 31, 2019 14:40
@alexander-schranz
Copy link
Contributor Author

@mamazu Thank you :)

@mamazu
Copy link
Member

mamazu commented Jul 31, 2019

Thanks for the contribution.

mamazu added a commit that referenced this pull request Aug 1, 2019
…ander-schranz)

This PR was merged into the 1.0-dev branch.

Discussion
----------

This fixes the tests on master branch after #502 / #503 where merged into it.

Commits
-------

5170c61 Fix product variant view factory original price test
mamazu added a commit that referenced this pull request Sep 21, 2019
This PR was merged into the 1.0-dev branch.

Discussion
----------

Reverts #503

Commits
-------

6ac03a5 Revert "Add onHand to product response"
2f96763 Fixing the spec tests
mamazu added a commit that referenced this pull request Feb 3, 2020
…er-schranz, mamazu)

This PR was merged into the 1.0-dev branch.

Discussion
----------

I misinterpredted how sylius checks the availibility of a product when implementing #503. 

This will now use the correct [availibilityChecker](https://github.com/Sylius/Sylius/blob/v1.5.3/src/Sylius/Component/Inventory/Checker/AvailabilityChecker.php) service of sylius.

Commits
-------

93f2a2c Add available property to ProductVariantView
e695674 Adapt phpunit tests for available flag
5a6b696 Rebasing to the new master and fixing tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants