shopware / SwagExtensionStore

The integrated Shopware Store for Shopware 6
2 stars 2 forks source link

EXS-127 - Listing modal for IAP #26

Closed Bird87ZA closed 2 months ago

Bird87ZA commented 2 months ago

What?

IAP Listing modal was implemented to show available IAP's

Why?

For customers who want to see what additional features they can purchase via IAP's.

Screenshots

Link in detail card when there are IAP's available image

Link in detail card when there are no IAP's image

Modal open image

Modal open (full window) image

Modal open with feature description visible image

Modal open with feature description visible (full window) image

Modal open with feature pricing visible image

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 70.87%. Comparing base (e0eb057) to head (016ddf6).

Files Patch % Lines
...sion-store/page/sw-extension-store-detail/index.js 66.66% 2 Missing :warning:
src/Services/InAppPurchasesService.php 0.00% 2 Missing :warning:
...-app-purchases/service/in-app-purchases.service.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## next-36303/feature/in-app-purchases #26 +/- ## ========================================================================= + Coverage 70.71% 70.87% +0.15% - Complexity 50 54 +4 ========================================================================= Files 32 32 Lines 881 903 +22 Branches 124 125 +1 ========================================================================= + Hits 623 640 +17 - Misses 229 234 +5 Partials 29 29 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

annikapantring commented 2 months ago

Hey Albert! Sorry I didn't mention this yesterday. Just double checked it now. Can you check the text styling again? My design components differ in the body text (feature description) and the feature name compared to your used text styles :)

Bird87ZA commented 2 months ago

I have updated the styling. There was 1 styling component missing, so it seems to be better now. Let me know :)

image