Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix builds#13244

Merged
lchrusciel merged 2 commits into
Sylius:1.9from
4c0n:fix-builds
Oct 27, 2021
Merged

Fix builds#13244
lchrusciel merged 2 commits into
Sylius:1.9from
4c0n:fix-builds

Conversation

@4c0n
Copy link
Copy Markdown
Contributor

@4c0n 4c0n commented Oct 22, 2021

Q A
Branch? 1.9 1.10
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
License MIT

@4c0n 4c0n requested a review from a team as a code owner October 22, 2021 13:37
@lchrusciel lchrusciel merged commit ff98f94 into Sylius:1.9 Oct 27, 2021
@lchrusciel
Copy link
Copy Markdown
Contributor

Thanks, Youri! 🥇

lchrusciel added a commit that referenced this pull request Oct 29, 2021
This PR was merged into the 1.9-dev branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | master
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | 
| License         | MIT

Should not be required after #13244 (in fact, the build is broken right now with it)

Commits
-------

8147bb1 Remove conflict with fos/rest-bundle
GSadee added a commit that referenced this pull request Nov 9, 2021
…responses to fix the build (GSadee)

This PR was merged into the 1.9 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.9
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | revert changes from #13244
| License         | MIT

<!--
 - Bug fixes must be submitted against the 1.9 or 1.10 branch (the lowest possible)
 - Features and deprecations must be submitted against the master branch
 - Make sure that the correct base branch is set

 To be sure you are not breaking any Backward Compatibilities, check the documentation:
 https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->


Commits
-------

caf9f0d [API] Revert changes of checked keys in cart and checkout responses to fix the build
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