Skip to content

fix update developer mode initialization#4073

Merged
mdmohsin7 merged 1 commit intomainfrom
fix/developer-page
Jan 5, 2026
Merged

fix update developer mode initialization#4073
mdmohsin7 merged 1 commit intomainfrom
fix/developer-page

Conversation

@krushnarout
Copy link
Member

will fix this crash too when opening developer page

flutter: ----------------FIREBASE CRASHLYTICS----------------
flutter: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
Consider canceling any active work during "dispose" or using the "mounted" getter to determine if the State is still active.
flutter: 
#0      State.context.<anonymous closure> (package:flutter/src/widgets/framework.dart:952:9)
#1      State.context (package:flutter/src/widgets/framework.dart:958:6)
#2      _DeveloperSettingsPageState.initState.<anonymous closure> (package:omi/pages/settings/developer.dart:42:7)

@krushnarout krushnarout requested a review from mdmohsin7 January 5, 2026 14:38
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

The pull request refactors the initialization of the DeveloperModeProvider. Previously, some boolean flags were initialized in the provider's constructor, and the initialize() method was called separately in developer.dart. The changes move all such initializations from the constructor into the initialize() method within developer_mode_provider.dart. Consequently, the DeveloperModeProvider is now initialized by calling its initialize() method immediately upon creation in main.dart, and the redundant initialize() call in developer.dart's initState has been removed.

@mdmohsin7 mdmohsin7 merged commit 8e4a3d1 into main Jan 5, 2026
1 check passed
@mdmohsin7 mdmohsin7 deleted the fix/developer-page branch January 5, 2026 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants