-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[TaxationBundle] DelegatingCalculator use the ServiceRegistry #2897
Conversation
<service id="sylius.form.type.tax_calculator_choice" class="%sylius.form.type.tax_calculator_choice.class%"> | ||
<argument>%sylius.tax_calculators%</argument> | ||
<tag name="form.type" alias="sylius_tax_calculator_choice" /> | ||
</service> | ||
|
||
<service id="sylius.tax_calculator" class="%sylius.tax_calculator.class%" /> | ||
<service id="sylius.tax_calculator" class="%sylius.tax_calculator.class%"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't it better to use sylius.registry.tax_calculator
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
d5ae365
to
96d1795
Compare
@pjedrzejewski @aramalipoor updated ! |
96d1795
to
e8f0ffc
Compare
@pjedrzejewski I will revert hte deletion of the |
bbf1414
to
745878b
Compare
@Sylius/core-team / @michalmarcinkowski any feedback ? |
745878b
to
678044e
Compare
@pjedrzejewski any feedback ? |
ping @pjedrzejewski |
[TaxationBundle] DelegatingCalculator use the ServiceRegistry
How many pings it takes to get something merged? Answer: Ping! 💃 Thank you Arnaud! |
[TaxationBundle] DelegatingCalculator use the ServiceRegistry
[TaxationBundle] DelegatingCalculator use the ServiceRegistry
@Sylius/core-team what do you think ?