@sunbiz can you please review this?

Satyam Jha (ba772316) at 08 Mar 14:04

Related Issue:

Fixes #143

Summary:

Fixes the non-functional "Remember Me" checkbox on the login screen. Previously, checking the box had no effect — credentials were never persisted and the login fields were never prefilled on subsequent launches. This PR wires up the full flow: LoginEvent now carries rememberMe and userType flags, AuthenticationRepository.loginUser saves credentials to an encrypted Hive box when the flag is set, and HiveStorageRepository stores the user type alongside the credentials so that prefilling is scoped to the correct login screen (Care Provider vs. Family Member). On screen load, LoginScreen.initState reads the saved credentials and prefills the fields only when the stored user type matches. Tests and regenerated mocks are updated to reflect the new method signatures.

Recording:

Satyam Jha (a98cee6c) at 08 Mar 06:25

Related Issue:

Fixes #142

Summary:

The onTap handler on home screen alert cards was a stub that only logged to the console. This wires it up to navigate to the correct vitals screen based on the alert's VitalType:

• VitalType.temperature → TemperatureScreen
• VitalType.heartRate → HeartRateScreen
• VitalType.respiratoryRate → RespirationRateScreen
• VitalType.bloodOxygen → SpO2RateScreen

Uses the existing pushScreen() pattern from the codebase.

Satyam Jha (7c31e1b3) at 07 Mar 19:41

Related Issue:

• Fixes #139

Description:

• Removed the outer Scaffold + bottomNavigationBar wrapper that was incorrectly constraining PersistentTabView's layout
• PersistentTabView is now returned directly as the root widget from MainNavigationScreen, which is the intended usage per the package's documentation
• Moved SystemUiOverlayStyle from the phantom AppBar to initState via SystemChrome.setSystemUIOverlayStyle(...) — status bar colour is preserved
• Added navBarOverlap: const NavBarOverlap.none() to prevent the nav bar from overlapping screen content

Testing:

• Verified scroll works on home screen with multiple alerts
• Verified status bar colour remains correct
• Verified bottom nav bar switching still works across all tabs

Recording:

Satyam Jha (4f728083) at 02 Mar 21:06

Satyam Jha (a3c51f4f) at 02 Mar 19:42

... and 1 more commit

@sunbiz pinging just to notify you.

@sunbiz This issue has been open for a long time.

What is the scope of this issue? Has anything changed in a year? What are your expectations? Please do let me know.

Great. but dependency_overrides is also a workaround which should be avoided, instead we should look for better solutions to this issue in future.

@sunbiz I have addressed all the issues and the pipeline is also running fine now, please review it once.

Satyam Jha (87237ea6) at 27 Feb 22:00

... and 6 more commits

Satyam Jha (a968a81d) at 27 Feb 21:43

Satyam Jha (960e89f6) at 27 Feb 21:37

@sunbiz

A few possible longer-term solutions:

1. Upgrade hive_generator to a newer version that's compatible with the latest analyzer (I explored this option, but hive_generator is not being maintained for almost 2+ years).
2. Pin a compatible analyzer version using dependency_overrides (though this can be fragile)
3. Migrate from Hive to another local storage solution (e.g., isar, drift) if the project is open to that.

Please let me know what are your thoughts on this issue