-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Closed
Labels
DailyKSv2KSv2EngineeringImprovementItem broken or needs improvement.Item broken or needs improvement.InternalRequires API changes or must be handled by Expensify staffRequires API changes or must be handled by Expensify staffReviewingHas a PR in reviewHas a PR in review
Description
Problem
Users can't enter HybridApp twice with the same deeplink. After navigating to another screen, putting the app in background and clicking on the same deeplink we weren't navigated to the correct place.
Solution
It's because initialURL didn't change and useEffect wasn't triggered. We can fix this behaviour by removing the broken useEffect and moving navigation logic to InitialURLContextProvider where we handle HybridApp urls
Reactions are currently unavailable
Metadata
Metadata
Labels
DailyKSv2KSv2EngineeringImprovementItem broken or needs improvement.Item broken or needs improvement.InternalRequires API changes or must be handled by Expensify staffRequires API changes or must be handled by Expensify staffReviewingHas a PR in reviewHas a PR in review