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

feat(Bonus Pagamenti Digitali): [#175265955] Bpd payment method refinement #2364

Merged
merged 55 commits into from
Nov 13, 2020

Conversation

fabriziofff
Copy link
Contributor

@fabriziofff fabriziofff commented Nov 10, 2020

This pr depends on #2360

Short description

This pr finalizes the graphical representation of payment methods in the bpd details.

wallet.length === 0 bpd is not activated on any method notActivable methods (no capability or activated by someone else

overview with FAKE data, the not activable will have the (!)

List of changes proposed in this pull request

  • Added PaymentMethodGroupedList in order to render the payment method toggle grouped by activable, not activable, activate on other channel.
  • Renamed the old PaymentMethodList.tsx to PaymentMethodRawList.tsx.
  • Changed WalletPaymentMethodBpdList in order to display the required warnings.
  • Added entrypoint "add payment method" to `WalletPaymentMethodBpdList.
  • Added atLeastOnePaymentMethodHasBpdEnabledSelector to know if there is at least one active payment method.
  • Added walletV2WithActivationStatusSelector in order to provide ready data to PaymentMethodGroupedList

How to test

Describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

fabriziofff and others added 30 commits November 9, 2020 19:03
…ansactionDetailComponent.tsx

Co-authored-by: Matteo Boschi <[email protected]>
…ansactionDetailComponent.tsx

Co-authored-by: Matteo Boschi <[email protected]>
fabriziofff and others added 21 commits November 11, 2020 14:30
…om:pagopa/io-app into 175266668-unsubscribe-bpd
…gopa/io-app into 175265955-bpd-payment-method-refinement
@Undermaken Undermaken merged commit 82e58cf into master Nov 13, 2020
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.

4 participants