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): [#175266668] Unsubscribe from bpd #2355

Merged
merged 28 commits into from
Nov 11, 2020

Conversation

fabriziofff
Copy link
Contributor

@fabriziofff fabriziofff commented Nov 9, 2020

This pr depends on #2354

Short description

This pr adds the complete unsubscription experience for the user.

Success Failure

Compared to the version displayed, the opacity has been increased and the cancel cashback button has been standardized to the design

List of changes proposed in this pull request

  • Changed BottomSheetContent.tsx to handle a footer (temp solution).
  • Changed BpdDetailsScreen.tsx in order to handle the loading, error, success of unsubscription
  • Added UnsubscribeComponent.tsx as bottomsheet content.
  • Changed UnsubscribeToBpd.tsx in order to handle the bottomsheet / unsubscription confirm.
  • Added unsubscription: RemoteValue<true, Error>; to the store in order to know the outcome of the unsubscription.
  • Added bottomSheetRawConfig to configure a bottomsheet with raw content (needed to add a footer).

@pagopa-github-bot pagopa-github-bot changed the title [#175266668] Unsubscribe from bpd feat(Bonus Pagamenti Digitali): [#175266668] Unsubscribe from bpd Nov 9, 2020
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Nov 9, 2020

Affected stories

  • 🌟 #175266668: Come CIT voglio potermi disiscrivere dal bpd all'interno della schermata di dettaglio

Generated by 🚫 dangerJS against 071483a

@codecov-io
Copy link

codecov-io commented Nov 9, 2020

Codecov Report

Merging #2355 (453b1b2) into master (9b3e602) will increase coverage by 0.00%.
The diff coverage is 57.83%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #2355    +/-   ##
========================================
  Coverage   48.91%   48.92%            
========================================
  Files         633      634     +1     
  Lines       17767    17829    +62     
  Branches     3210     3505   +295     
========================================
+ Hits         8691     8722    +31     
- Misses       9034     9065    +31     
  Partials       42       42            
Impacted Files Coverage Δ
ts/components/bottomSheet/BottomSheetContent.tsx 100.00% <ø> (ø)
ts/components/core/typography/H3.tsx 100.00% <ø> (ø)
ts/components/core/typography/Label.tsx 100.00% <ø> (ø)
ts/utils/bottomSheet.ts 43.75% <33.33%> (-2.41%) ⬇️
...nus/bpd/store/reducers/details/activation/index.ts 48.88% <37.50%> (-6.29%) ⬇️
...res/bonus/bpd/screens/details/BpdDetailsScreen.tsx 63.63% <42.85%> (-29.70%) ⬇️
...Vacanze/components/buttons/ButtonConfigurations.ts 55.55% <50.00%> (-1.59%) ⬇️
...etails/components/unsubscribe/UnsubscribeToBpd.tsx 61.53% <61.53%> (ø)
...ls/components/unsubscribe/UnsubscribeComponent.tsx 100.00% <100.00%> (ø)
...us/bpd/screens/details/periods/BpdActivePeriod.tsx 100.00% <100.00%> (ø)
... and 4 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 9b3e602...453b1b2. Read the comment docs.

@fabriziofff fabriziofff marked this pull request as ready for review November 10, 2020 12:46
@Undermaken Undermaken merged commit a051b09 into master Nov 11, 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