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

chore: [IOPID-1298,IOPID-1396] Main stack navigator - startup saga synch #5420

Merged
merged 5 commits into from
Jan 24, 2024

Conversation

shadowsheep1
Copy link
Member

Short description

This PR tries to synchronize the Main stack navigator with the startup saga business logic.

List of changes proposed in this pull request

It uses the waitForMainNavigator generator function that acts like waitForNavigatorServiceInitialization.

How to test

  • Run the app and during a first onboarding, before reaching the pin creation screen, re-launch the app. You'd navigate to the last screen where you was before re-launching the app.
  • Try different login flows, you'd be reach the messages home all the times.

@shadowsheep1 shadowsheep1 self-assigned this Jan 22, 2024
@pagopa-github-bot pagopa-github-bot changed the title [IOPID-1298,IOPID-1396] Main stack navigator - startup saga synch chore: [IOPID-1298,IOPID-1396] Main stack navigator - startup saga synch Jan 22, 2024
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jan 22, 2024

Affected stories

  • ⚙️ IOPID-1298: [APP] - Analizzare e migliorare la sincronizzazione tra il cambio del main stack navigator e la saga di startup
  • ⚙️ IOPID-1396: [APP] implementazione sincronizzazione startup saga/stack navigator
    subtask of
    • ⚙️ IOPID-1298: [APP] - Analizzare e migliorare la sincronizzazione tra il cambio del main stack navigator e la saga di startup

Generated by 🚫 dangerJS against 77686e6

@shadowsheep1 shadowsheep1 added the IO-A&I IO - Autenticazione e Identità label Jan 22, 2024
@shadowsheep1 shadowsheep1 changed the title chore: [IOPID-1298,IOPID-1396] Main stack navigator - startup saga synch chore: [IOPID-1298,IOPID-1396] Main stack navigator - startup saga synch Jan 22, 2024
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (9634f37) 47.48% compared to head (85a62ed) 47.46%.

❗ Current head 85a62ed differs from pull request most recent head 77686e6. Consider uploading reports for the commit 77686e6 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5420      +/-   ##
==========================================
- Coverage   47.48%   47.46%   -0.03%     
==========================================
  Files        1604     1604              
  Lines       33577    33600      +23     
  Branches     8197     8204       +7     
==========================================
+ Hits        15944    15948       +4     
- Misses      17575    17594      +19     
  Partials       58       58              
Files Coverage Δ
ts/navigation/NavigationService.ts 75.86% <75.00%> (-0.14%) ⬇️
ts/navigation/AppStackNavigator.tsx 46.93% <12.50%> (-5.45%) ⬇️
ts/sagas/startup.ts 29.44% <0.00%> (-2.11%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9634f37...77686e6. Read the comment docs.

@shadowsheep1 shadowsheep1 added the legal-review-needed ⚖️ ⛔ this PR is under legal check. label Jan 22, 2024
@shadowsheep1 shadowsheep1 marked this pull request as ready for review January 22, 2024 15:12
@shadowsheep1 shadowsheep1 requested a review from a team as a code owner January 22, 2024 15:12
@shadowsheep1 shadowsheep1 added legal-approved ⚖️ ✅ this PR has legal compliance and removed legal-review-needed ⚖️ ⛔ this PR is under legal check. labels Jan 23, 2024
Copy link
Contributor

@Vangaorth Vangaorth left a comment

Choose a reason for hiding this comment

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

LGTM

@shadowsheep1 shadowsheep1 merged commit 71d6507 into master Jan 24, 2024
5 checks passed
@shadowsheep1 shadowsheep1 deleted the IOPID-1298-stack-navigator-synch branch January 24, 2024 08:19
Vangaorth added a commit that referenced this pull request Jan 24, 2024
…#5429)

This PR depends on #5420 

## Short description
This PR fixes the E2E tests, following changes from the stack navigator
and startup saga sync.

## List of changes proposed in this pull request
- reloadReactNative replaced with launchApp({ newInstance: true}) to
prevent bad behaviours
- improved a credit card test by searching for a testId instead of a
text
- better system keyboard closing

## How to test
Run the E2E tests.

---------

Co-authored-by: Fabio Bombardi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO-A&I IO - Autenticazione e Identità legal-approved ⚖️ ✅ this PR has legal compliance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants