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

Skip to content

[Minor] Update service definition to not using depracated service#11982

Merged
Zales0123 merged 1 commit into
Sylius:1.8from
lchrusciel:depracted-service
Nov 2, 2020
Merged

[Minor] Update service definition to not using depracated service#11982
Zales0123 merged 1 commit into
Sylius:1.8from
lchrusciel:depracted-service

Conversation

@lchrusciel
Copy link
Copy Markdown
Contributor

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

According to note:
image

Zales0123 added a commit that referenced this pull request Nov 2, 2020
…d service (lchrusciel)

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

Same as #11982, but this service does not exist in 1.8

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

570a312 [Minor] Update service definition to not using depracated service
@Zales0123 Zales0123 added the Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). label Nov 2, 2020
@Zales0123 Zales0123 merged commit 5d34999 into Sylius:1.8 Nov 2, 2020
@Zales0123
Copy link
Copy Markdown
Contributor

Thanks, Łukasz! 🥇

@lchrusciel lchrusciel deleted the depracted-service branch November 2, 2020 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants