Skip to content
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 defining custom amounts for gift cards #154

Merged
merged 12 commits into from
Sep 15, 2021

Conversation

Roshyo
Copy link
Contributor

@Roshyo Roshyo commented Sep 10, 2021

Also allow defining custom message

Also allow defining custom message
@codecov
Copy link

codecov bot commented Sep 10, 2021

Codecov Report

Merging #154 (9efa17e) into 0.12.x (ad0ab41) will increase coverage by 8.06%.
The diff coverage is 49.55%.

Impacted file tree graph

@@             Coverage Diff              @@
##             0.12.x     #154      +/-   ##
============================================
+ Coverage     26.55%   34.61%   +8.06%     
- Complexity      399      436      +37     
============================================
  Files            77       84       +7     
  Lines          1303     1404     +101     
============================================
+ Hits            346      486     +140     
+ Misses          957      918      -39     
Impacted Files Coverage Δ
src/Form/Extension/AddToCartTypeExtension.php 0.00% <0.00%> (ø)
src/Form/Extension/ProductTypeExtension.php 0.00% <0.00%> (ø)
src/Form/Type/AddToCartGiftCardInformationType.php 0.00% <0.00%> (ø)
src/Factory/GiftCardFactory.php 100.00% <100.00%> (+100.00%) ⬆️
src/Model/GiftCard.php 70.32% <100.00%> (+27.80%) ⬆️
src/Model/OrderItemTrait.php 100.00% <100.00%> (ø)
src/Model/ProductTrait.php 100.00% <100.00%> (+100.00%) ⬆️
src/Operator/OrderGiftCardOperator.php 84.00% <100.00%> (+84.00%) ⬆️
src/Order/AddToCartCommand.php 100.00% <100.00%> (ø)
src/Order/Factory/AddToCartCommandFactory.php 100.00% <100.00%> (ø)
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad0ab41...9efa17e. Read the comment docs.

Copy link
Member

@loevgaard loevgaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First round. Looking good 🎉

src/Order/Dto/AddGiftCardToCartInformationInterface.php Outdated Show resolved Hide resolved
src/Order/Dto/AddGiftCardToCartInformationInterface.php Outdated Show resolved Hide resolved
src/Order/Factory/AddToCartCommandFactory.php Show resolved Hide resolved
src/Resources/config/services/form.xml Show resolved Hide resolved
src/Resources/config/services/order.xml Show resolved Hide resolved
src/Resources/translations/messages.da.yml Outdated Show resolved Hide resolved
src/Resources/translations/messages.da.yml Outdated Show resolved Hide resolved
src/Resources/translations/messages.da.yml Outdated Show resolved Hide resolved
Copy link
Member

@loevgaard loevgaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few last comments :)

src/Order/AddToCartCommandInterface.php Show resolved Hide resolved
src/Order/Factory/AddToCartCommandFactory.php Outdated Show resolved Hide resolved
src/Form/Extension/AddToCartTypeExtension.php Outdated Show resolved Hide resolved
src/Order/GiftCardInformation.php Outdated Show resolved Hide resolved
src/Order/GiftCardInformation.php Outdated Show resolved Hide resolved
Remove un-needed methods signatures

Remove un-needed default null property
Add Upgrade guide

Remove un-needed template override
@Roshyo Roshyo merged commit 0a04af0 into 0.12.x Sep 15, 2021
@Roshyo Roshyo deleted the enh-configurable-gift-cards branch September 15, 2021 07:54
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.

2 participants