Fix feature flags silent failure and profile page infinite loading by Mozart299 · Pull Request #3679 · airqo-platform/AirQo-frontend

Mozart299 · 2026-06-24T18:29:36Z

Add ?? false to isFeatureEnabled() call to prevent TypeError when PostHog returns null on fresh login with no local cache, which was silently disabling all feature flags for the entire session
Call reloadFlags() at startup after PostHog init so flags load for unauthenticated/guest users from first launch
Dispatch LoadUser in ProfilePage initState to retry if user data isn't already loaded, and add Retry button on UserLoadingError state

Summary by CodeRabbit

New Features
- Feature flags are refreshed automatically during app startup for more up-to-date experiences.
- The profile screen now offers a Retry button when user data fails to load.
Bug Fixes
- Improved profile behavior: user data now re-fetches when the screen opens in a non-loaded state.
- Made feature flag handling more reliable by treating unavailable flag results as false.

- Add ?? false to isFeatureEnabled() call to prevent TypeError when PostHog returns null on fresh login with no local cache, which was silently disabling all feature flags for the entire session - Call reloadFlags() at startup after PostHog init so flags load for unauthenticated/guest users from first launch - Dispatch LoadUser in ProfilePage initState to retry if user data isn't already loaded, and add Retry button on UserLoadingError state

coderabbitai · 2026-06-24T18:29:57Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d9ccee20-b739-4c9d-aa87-4a1ec766b694

📥 Commits

Reviewing files that changed from the base of the PR and between 66d26e2 and e20f10a.

📒 Files selected for processing (1)

src/mobile/lib/src/app/profile/pages/profile_page.dart

🚧 Files skipped from review as they are similar to previous changes (1)

src/mobile/lib/src/app/profile/pages/profile_page.dart

📝 Walkthrough

Walkthrough

The app now reloads feature flags during startup after PostHog setup, normalizes null flag checks to false, and updates the profile page to load user data on entry and retry from the error state.

Changes

Startup and profile flow updates

Layer / File(s)	Summary
Feature flags reload after setup `src/mobile/lib/main.dart`, `src/mobile/lib/src/app/shared/services/feature_flag_service.dart`	`main.dart` imports `FeatureFlagService` and awaits `reloadFlags()` after PostHog setup. `reloadFlags()` now stores `false` when the SDK returns `null`.
Profile user load and retry `src/mobile/lib/src/app/profile/pages/profile_page.dart`	`initState` dispatches `LoadUser()` when the bloc is not already loaded or loading, and the error UI now includes a Retry button that dispatches `LoadUser()`.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

airqo-platform/AirQo-frontend#3541: Uses the same FeatureFlagService state that is refreshed during startup here.

Suggested reviewers

Baalmart

Poem

🚩 PostHog wakes, the flags roll in,
Null turns false, and state stays thin.
Profile loads, and if it sighs,
Retry clicks bring better skies.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the two main fixes: feature flag initialization and the profile page loading issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/feature-flags-fresh-login

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)

JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/mobile/lib/src/app/profile/pages/profile_page.dart`:
- Around line 22-30: Avoid triggering a duplicate user fetch from
profile_page.dart’s initState in ProfilePage: the current state check only
excludes UserLoaded, so UserLoading still falls through and dispatches
LoadUser() again. Update the condition to also skip dispatch when the UserBloc
state is already loading, using the existing UserBloc/UserLoaded/UserLoading
flow in initState so only one load can be enqueued at a time.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 65312af2-7390-4ddc-bddf-ea14fab78fb0

📥 Commits

Reviewing files that changed from the base of the PR and between a84056e and 66d26e2.

⛔ Files ignored due to path filters (1)

src/mobile/pubspec.lock is excluded by !**/*.lock

📒 Files selected for processing (3)

src/mobile/lib/main.dart
src/mobile/lib/src/app/profile/pages/profile_page.dart
src/mobile/lib/src/app/shared/services/feature_flag_service.dart

… loading

Copilot

Pull request overview

This PR improves reliability of PostHog feature flags on fresh installs/logins and prevents the Profile page from getting stuck in a non-loaded state by proactively triggering user-load and offering a retry action.

Changes:

Make feature-flag evaluation resilient to null results from PostHog and trigger a flags refresh right after PostHog initialization.
On ProfilePage, dispatch LoadUser during initState when the user state isn’t already loading/loaded.
Add a Retry button to the profile error UI.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/mobile/pubspec.lock	Updates dependency resolution; also bumps the lockfile’s minimum Dart/Flutter SDK versions.
src/mobile/lib/src/app/shared/services/feature_flag_service.dart	Defaults feature flags to `false` when PostHog returns `null`.
src/mobile/lib/main.dart	Reloads feature flags immediately after PostHog setup.
src/mobile/lib/src/app/profile/pages/profile_page.dart	Triggers user reload on page init and adds Retry UI on load failure.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

coderabbitai Bot reviewed Jun 24, 2026

View reviewed changes

Comment thread src/mobile/lib/src/app/profile/pages/profile_page.dart

Guard ProfilePage initState against duplicate LoadUser dispatch while…

67c1d24

… loading

Mozart299 added the mobile-app label Jun 24, 2026

Mozart299 requested a review from Baalmart June 24, 2026 18:37

Baalmart requested a review from Copilot June 24, 2026 18:48

Copilot started reviewing on behalf of Baalmart June 24, 2026 18:48 View session

Copilot AI reviewed Jun 24, 2026

View reviewed changes

Comment thread src/mobile/lib/src/app/profile/pages/profile_page.dart Outdated

Potential fix for pull request finding

e20f10a

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Baalmart merged commit 47af504 into staging Jun 25, 2026
25 of 27 checks passed

Baalmart deleted the fix/feature-flags-fresh-login branch June 25, 2026 06:15

Baalmart mentioned this pull request Jun 25, 2026

move to production #3681

Merged

3 tasks

coderabbitai Bot mentioned this pull request Jul 8, 2026

Mobile posthog analytics #3769

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix feature flags silent failure and profile page infinite loading#3679

Fix feature flags silent failure and profile page infinite loading#3679
Baalmart merged 3 commits into
stagingfrom
fix/feature-flags-fresh-login

Mozart299 commented Jun 24, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jun 24, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Review ran into problems

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Mozart299 commented Jun 24, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Review ran into problems

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Mozart299 commented Jun 24, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 24, 2026 •

edited

Loading