openedx-unsupported / edx-app-android

The Open edX mobile app for Android!
https://docs.google.com/document/d/15MYZ2LAmahB82P5wkPkQOqjszfou0Qkeae3qz7STxUQ/edit
Apache License 2.0
301 stars 333 forks source link

fix: App crash on primary email change from web #1848

Closed farhan-arshad-dev closed 9 months ago

farhan-arshad-dev commented 9 months ago

Description

LEARNER-9831

How to test

Acceptance Criteria

codecov[bot] commented 9 months ago

Codecov Report

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

Comparison is base (2b52504) 1.07% compared to head (dcdd5bb) 1.07%.

Files Patch % Lines
...tp/authenticator/OauthRefreshTokenAuthenticator.kt 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1848 +/- ## ======================================== Coverage 1.07% 1.07% Complexity 137 137 ======================================== Files 538 538 Lines 26326 26325 -1 Branches 3395 3395 ======================================== Hits 284 284 + Misses 26015 26014 -1 Partials 27 27 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

miankhalid commented 9 months ago

@farhan-arshad-dev great work in doing an in-depth discovery and finding the fix for this pesky issue 👏 🌟