-
Notifications
You must be signed in to change notification settings - Fork 43
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
Allow to use Sylius from 1.7 to 1.10 and php 7.4 and 8.0 #139
Conversation
Codecov Report
@@ Coverage Diff @@
## master #139 +/- ##
=============================================
+ Coverage 9.51% 26.55% +17.03%
- Complexity 360 399 +39
=============================================
Files 63 77 +14
Lines 1166 1303 +137
=============================================
+ Hits 111 346 +235
+ Misses 1055 957 -98
Continue to review full report at Codecov.
|
@@ -28,13 +28,13 @@ | |||
<service id="setono_sylius_gift_card.controller.action.gift_card_balance" | |||
class="Setono\SyliusGiftCardPlugin\Controller\Action\GiftCardBalanceAction"> | |||
<argument type="service" id="setono_sylius_gift_card.repository.gift_card"/> | |||
<argument type="service" id="fos_rest.view_handler.default"/> | |||
<argument type="service" id="twig"/> |
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.
When we release we need to keep in mind that this is a BC break
Remove Assert in Extension and use @var
Export Download gift card PDF to API + tests
Expose Customer GiftCards in shop
Expose Gift card balance
Co-authored-by: Joachim Løvgaard <joachim@loevgaard.dk>
Co-authored-by: Joachim Løvgaard <joachim@loevgaard.dk>
…e-add-gc-to-order
Expose Add/Remove GC to/from order
Add first Behat scenario for API
update template according to latest changes in sylius
Allow to create Product GC from API
No description provided.