Merge back parent release#3640
Conversation
refs: MBL-19940 affects: Student release note: Fixed an issue where students were logged out when entering split-screen or multi-window mode on Android tablets. (cherry picked from commit fc199e6)
🧪 Unit Test Results📊 Summary
Last updated: Mon, 13 Apr 2026 07:50:38 GMT |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Summary
This is a small, focused patch release with two changes.
Version bump (apps/parent/build.gradle): Routine 4.11.0 → 4.11.1 patch bump.
FeatureFlagProvider.fetchEnvironmentFeatureFlags: Removes shouldIgnoreToken = true from the RestParams. This is the right fix — the flag was causing the GET /api/v1/features/environment request to be sent without an Authorization header, even though the function is always invoked post-authentication (in SplashViewModel, CallbackActivity, and InitActivity) and stores results keyed by the logged-in user's ID. The corrected behavior is also consistent with how FeaturesManager.getEnvironmentFeatureFlags has always called the same endpoint.
Issues
- Test doesn't verify the exact
RestParams—libs/pandautils/src/test/java/com/instructure/pandautils/utils/FeatureFlagProviderTest.kt:142: Theany()matcher means the test won't catch future changes toRestParams(e.g., accidentally re-addingshouldIgnoreToken = trueor droppingisForceReadFromNetwork = true). See inline comment for a suggested tighter assertion.
📊 Code Coverage Report✅ Student
✅ Teacher
✅ Pandautils
📈 Overall Average
|
2 participants
No description provided.