Skip to content

Fix timezone handling and update deprecated API calls#5

Merged
shceo merged 2 commits into
mainfrom
claude/fix-prayer-profile-issues-MKAek
Mar 15, 2026
Merged

Fix timezone handling and update deprecated API calls#5
shceo merged 2 commits into
mainfrom
claude/fix-prayer-profile-issues-MKAek

Conversation

@ahksm

@ahksm ahksm commented Mar 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR addresses several API deprecations and improves code clarity across the notification, location, and profile services.

Key Changes

  • Notification Service: Fixed timezone initialization by directly using the timezone name string instead of accessing a non-existent identifier property on the FlutterTimezone response
  • Location Service: Updated Geolocator.getCurrentPosition() to use the new locationSettings parameter instead of the deprecated desiredAccuracy parameter
  • Profile Screen: Replaced FaIcon (FontAwesome icon) with standard Icon widget in the action row component

Implementation Details

  • The timezone fix simplifies the code by removing an unnecessary property access that was causing issues
  • The location service update aligns with the latest Geolocator package API, which now uses a LocationSettings object for configuration
  • The icon widget change standardizes the UI component to use Flutter's built-in icon system instead of FontAwesome

https://claude.ai/code/session_016nf1sAjbEyPV2bR97cx1iw

claude added 2 commits March 15, 2026 09:36
- Fix NotificationService: FlutterTimezone.getLocalTimezone() returns
  String, not object with .identifier property
- Fix LocationService: update getCurrentPosition() for geolocator v10
  API (desiredAccuracy → locationSettings)
- Fix ProfileScreen: replace FaIcon with Icon in _ActionRow to support
  both Material and FontAwesome icons (FaIcon crashes on Material icons)

https://claude.ai/code/session_016nf1sAjbEyPV2bR97cx1iw
Drift's NativeDatabase requires the native SQLite library bundled in
the APK. Without sqlite3_flutter_libs, Android fails with:
"failed to load dynamic library libsqlite3.so"

Also call applyWorkaroundToOpenSqlite3OnOldAndroidVersions() to ensure
compatibility across all Android versions.

https://claude.ai/code/session_016nf1sAjbEyPV2bR97cx1iw
@shceo shceo merged commit ff4af21 into main Mar 15, 2026
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.

3 participants