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): [#174816293,#174796530,#174794127] Display BDP in bonus list #2207

Merged
merged 8 commits into from
Sep 18, 2020

Conversation

Undermaken
Copy link
Contributor

@Undermaken Undermaken commented Sep 17, 2020

Short description

To accomplish this task multiple changes were needed.

IO App

List of changes proposed in this pull request

  • moved checkSession check outside of scoped saga (an optimization that doesn't refer this task)
  • moved bonus available request from bonus vacanze to content API (it is no longer specific for bonus vacanze)
  • change the bonus available endpoint to the new one if bdp is enabled
  • render all bonus available items that are not hidden
  • handle all bonus available taps that are mapped

IO services metadata

List of changes proposed in this pull request

IO dev server PR#28

  • support the new /session response (added token for bdp and myportal)
  • support the new API for bonus available

How to test

  • IO dev-server git pull;git checkout support-bpd;yarn generate:all;yarn start
  • IO app: connect the app to io-dev-server. Run this command yarn generate:all before launching the app

remove check session as duplicated check
moved bonus available request from bonus vacanze to content
bonus available won't be displayed if it is hidden
only hardoced bonus type can be handled
@pagopa-github-bot pagopa-github-bot changed the title [#174816293,#174796530] Display BDP in bonus list feat: [#174816293,#174796530] Display BDP in bonus list Sep 17, 2020
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Sep 17, 2020

Affected stories

  • 🌟 #174816293: Come CIT voglio poter selezionare "cashback pagamenti digitali" dalla lista dei bonus disponibili
  • 🌟 #174796530: Aggiungere al dev server il token di BPD rilasciato dalla getSession

Generated by 🚫 dangerJS against 852981b

@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2020

Codecov Report

Merging #2207 into master will increase coverage by 0.00%.
The diff coverage is 33.33%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2207   +/-   ##
=======================================
  Coverage   47.17%   47.17%           
=======================================
  Files         492      491    -1     
  Lines       14763    14750   -13     
  Branches     3005     3004    -1     
=======================================
- Hits         6964     6958    -6     
+ Misses       7757     7750    -7     
  Partials       42       42           
Impacted Files Coverage Δ
...ures/bonus/bonusVacanze/api/backendBonusVacanze.ts 40.00% <ø> (+1.11%) ⬆️
ts/sagas/wallet/pagopaApis.ts 16.44% <ø> (-0.12%) ⬇️
...onus/bonusVacanze/screens/AvailableBonusScreen.tsx 50.00% <25.00%> (-3.07%) ⬇️
...atures/bonus/bonusVacanze/store/sagas/bonusSaga.ts 68.18% <100.00%> (+1.51%) ⬆️

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 0832570...852981b. Read the comment docs.

@Undermaken Undermaken marked this pull request as ready for review September 17, 2020 17:10
@pagopa-github-bot pagopa-github-bot changed the title feat: [#174816293,#174796530] Display BDP in bonus list feat(Bonus Pagamenti Digitali): [#174816293,#174796530] Display BDP in bonus list Sep 17, 2020
…96530-bpd-available-bonus

# Conflicts:
#	ts/features/bonus/bonusVacanze/api/backendBonusVacanze.ts
#	ts/features/bonus/bonusVacanze/screens/AvailableBonusScreen.tsx
#	ts/features/bonus/bonusVacanze/store/sagas/bonusSaga.ts
#	ts/features/bonus/bonusVacanze/store/sagas/handleLoadAvailableBonuses.ts
#	ts/screens/wallet/WalletHomeScreen.tsx
@fabriziofff fabriziofff merged commit ff2c805 into master Sep 18, 2020
@fabriziofff fabriziofff deleted the 174816293-174796530-bpd-available-bonus branch September 18, 2020 08:42
@Undermaken Undermaken changed the title feat(Bonus Pagamenti Digitali): [#174816293,#174796530] Display BDP in bonus list feat(Bonus Pagamenti Digitali): [#174816293,#174796530,#174794127] Display BDP in bonus list Sep 18, 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