-
Notifications
You must be signed in to change notification settings - Fork 537
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 #5657: Handle App Crash When Changing to Landscape After Interacting with Previous Responses #5680
base: develop
Are you sure you want to change the base?
Fix #5657: Handle App Crash When Changing to Landscape After Interacting with Previous Responses #5680
Conversation
Coverage ReportResultsNumber of files assessed: 1 Exempted coverageFiles exempted from coverage
|
…thub.com/TanishMoral11/oppia-android into fix/handle-app-crash-on-landscape-change
Hey @adhiamboperes , PTAL |
Coverage ReportResultsNumber of files assessed: 2 Exempted coverageFiles exempted from coverage
|
Explanation
This PR addresses the issue where the Oppia Android app crashes when switching to landscape orientation after interacting with previous responses in the Exploration/Question player. The crash occurs due to uninitialized state when loading audio content. This fix ensures that the state is properly initialized before attempting to load the audio, preventing the app from crashing.
Changes Made:
Fixes #5657: App crashes when changing configuration to landscape after interacting with previous responses.
Essential Checklist
Before :-
https://github.com/user-attachments/assets/0ee8de2f-45f2-4b2a-af57-0a371b6f34b2
After :-
https://github.com/user-attachments/assets/3b3a3be8-b80c-43b2-ba68-ffa3aa03ed3d