echo-swent / echo

4 stars 0 forks source link

Fix null user profile bug #269

Closed octogradiste closed 6 months ago

octogradiste commented 6 months ago

This will close #258, the bug the coaches pointed out during the last scrum retrospective.

To solve it, I moved the logic from the NavigationActions to the AppNavHost. This allows us to do the checks at application start. I added a loading animation to avoid blocking the app during the checks. This will close #105.

Note that as the authentication relied on the NavigationActions for navigating to the correct route (profile creation or map), there were some tests which broke. So I had to put some more logic into the authentication view models.

sonarcloud[bot] commented 6 months ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
90.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud