-
Notifications
You must be signed in to change notification settings - Fork 254
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
fix: Corrige navegação ao ser direcionado via Deep Link #950
Conversation
Olá. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #950 +/- ##
==========================================
- Coverage 99.87% 99.52% -0.36%
==========================================
Files 34 8 -26
Lines 824 209 -615
==========================================
- Hits 823 208 -615
Misses 1 1 ☔ View full report in Codecov by Sentry. |
Fiz uma PoC com duas rotas, home e cart, para poder testar o acesso direto a rota cart. Gerei esse App, e publiquei no Firebase Hosting, e pode ser validado através da url de home: Aqui funcionou normlamente, não sei se seria algo nesse sentido ou se havia um cenário mais específico. |
Description
Este PR corrige a navegação quando é direcionado via Deep Link.
Conforme mencionado na issue #934, o arquivo
modular_route_information_parser.dart
tem uma variável para uma condição que verifica se é a primeira vez que é feito o parse. Porém não funciona corretamente quado o usuário sai do App e volta através de um deep link, que irá o redirecionar para uma rota interna.Por isso, acredito que seja melhor remover essa variável e a condição para solucionar este problema. Em contra partida o parse deverá ser feito sempre, porém, acho menos custoso que ter algum observador para identificar que o usuário 'suspendeu' e 'ativou' o app novamente.
Cenário Observado
Aplicativo em Flutter 3.19.1
Package Modular 6.3.2
Checklist
fix:
,feat:
,docs:
etc).docs
and added dartdoc comments with///
.examples
.Breaking Change
Related Issues
Fixes #934