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

[#173521040] Update elgibility expired screen & eligibility / activation navigation rework #1994

Merged
merged 16 commits into from
Jul 14, 2020

Conversation

fabriziofff
Copy link
Contributor

Short description:
This pr updates the eligibility expired screen and refactor the navigation for the eligibility and activation phase.

List of changes proposed in this pull request:

  • Graphical updates for EligibilityExpired.tsx
  • Added new cta in EligibilityExpired.tsx in order to restart the eligibility phase.
  • Rework navigation in order to avoid the dispatch of navigationPop when calling checkBonusVacanzeEligibility.request()
  • Restored the NavigationActions.back() when canceled or closed the eligibility / activation workflow.
  • Added cta behaviour for BonusAlreadyExists button -> onGoToBonus -> goto walletHome.

@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jun 30, 2020

Affected stories

  • 🌟 #173521040: Aggiornamento grafico schermata Eligibility Expired

Generated by 🚫 dangerJS

@Undermaken Undermaken marked this pull request as draft July 1, 2020 12:54
@fabriziofff fabriziofff marked this pull request as ready for review July 2, 2020 18:01
@fabriziofff fabriziofff marked this pull request as draft July 2, 2020 18:15
@fabriziofff fabriziofff marked this pull request as ready for review July 3, 2020 09:24
@Undermaken Undermaken requested a review from CrisTofani as a code owner July 14, 2020 13:01
@@ -0,0 +1,20 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

@CrisTofani we need you help: could it be possible to include this svg in our icon-pack?

Copy link
Contributor

Choose a reason for hiding this comment

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

@fabriziofff Since the png is used, is this svg needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Due to legacy configuration of react-native-typescript-transformer the svg file could not be imported (https://github.com/kristerkari/react-native-svg-transformer#installation-and-configuration) and was replaced by a png.
removed in e5af8d5

@Undermaken Undermaken merged commit 0a2b3aa into master Jul 14, 2020
@fabriziofff fabriziofff deleted the 173521040-update-elgibility-expired-screen branch January 21, 2022 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants