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

Skip to content

Update symfony lock file#11757

Closed
foxou33 wants to merge 8 commits into
Sylius:1.7from
foxou33:issue_11438
Closed

Update symfony lock file#11757
foxou33 wants to merge 8 commits into
Sylius:1.7from
foxou33:issue_11438

Conversation

@foxou33
Copy link
Copy Markdown
Contributor

@foxou33 foxou33 commented Aug 22, 2020

Q A
Branch? 1.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #11438
License MIT

@foxou33 foxou33 requested a review from a team as a code owner August 22, 2020 19:04
jacquesbh and others added 4 commits August 28, 2020 12:44
…acquesbh)

This PR was merged into the 1.7 branch.

Discussion
----------

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

This change (using the billing address instead of shipping address to calculate taxes) is actually a BIG change. It changed all our tax calculation and we didn't see that coming. 😭 

Commits
-------

2534e76 Add warning on tax calculation in 1.7's upgrade
@lchrusciel
Copy link
Copy Markdown
Contributor

The build will be green after #11780

lchrusciel and others added 3 commits September 1, 2020 11:45
…chrusciel)

This PR was merged into the 1.7 branch.

Discussion
----------

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

All fixes are related to psalm errors. This should fix 1.7 build.

This PR contains cherry-picked commit: Sylius@31671eb

<!--
 - Bug fixes must be submitted against the 1.7 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
-------

7fddcba [Maintenance] Adjust variable names to interfaces
d0fba24 [Maintenance][HotFix] Conflict with Symofny/Form 4.4.12
@lchrusciel
Copy link
Copy Markdown
Contributor

Hey @foxou33, thanks for your PR, but it was done already in #11810. Sorry for not taking your commit for that.

@lchrusciel lchrusciel closed this Sep 8, 2020
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