Skip to content

Migrate compass_app to SharedPreferencesAsync#2841

Open
harshyadavDeveloper wants to merge 1 commit intoflutter:mainfrom
harshyadavDeveloper:fix-compass-sharedpreferences-migration
Open

Migrate compass_app to SharedPreferencesAsync#2841
harshyadavDeveloper wants to merge 1 commit intoflutter:mainfrom
harshyadavDeveloper:fix-compass-sharedpreferences-migration

Conversation

@harshyadavDeveloper
Copy link
Copy Markdown

Description

Migrated SharedPreferences usage in compass_app to SharedPreferencesAsync to align with the latest shared_preferences recommendations.

Changes made

  • Replaced SharedPreferences.getInstance() usages
  • Updated async preference reads with awaited getters
  • Migrated token persistence logic
  • Updated test setup preference clearing logic

Fixes #2720

Pre-launch Checklist

@harshyadavDeveloper
Copy link
Copy Markdown
Author

Implemented the migration from SharedPreferences to SharedPreferencesAsync in compass_app as discussed in #2720.

The changes include:

  • replacing legacy SharedPreferences.getInstance() usage
  • updating async getters
  • migrating token persistence logic
  • updating related test setup code

Feedback and review are welcome. Thanks!

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.

[compass_app] replace SharedPreferences with SharedPreferencesWithCache (or SharedPreferencesAsync)

1 participant