Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: Consolidate Shared Preferences Keys #262

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

davidetarsi
Copy link

This pull request refactors the management of shared preferences by grouping all preference keys in a single location for better maintainability and consistency.

Changes made:

  • Grouped all shared preferences keys, including both normal keys and enabled keys, in the const.dart file for centralized management.
  • Did not update the hardcoded strings in sp.getString() calls to use the newly defined spKey constants to avoid potential regressions.

Benefits:

  • Improved code organization and maintainability by consolidating preference keys.
  • Simplified future updates, reducing the risk of inconsistencies.

Notes for Reviewers:

  • Updating the sp.getString() calls with the new constants is planned for a future update.
  • Please verify that no existing functionality has been impacted by the changes to key declarations.

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.

1 participant