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

Rework State Management #425

Merged
merged 1 commit into from
Jan 4, 2023
Merged

Rework State Management #425

merged 1 commit into from
Jan 4, 2023

Conversation

ricoberger
Copy link
Member

@ricoberger ricoberger commented Jan 2, 2023

Until now we used GetX for the state management in the app. Since we are not very happy with this, we decided to remove GetX in all places and use Provider for the state management. We also switched to the Navigator instead of using GetX for the routing in the app. Last but not least we also removed GetStorage and save the clusters, providers and app settings via the Flutter Secure Storage package.

This commit also updates the required Dart and Go dependencies.

Note: The dark mode is not working in this version, because there will be a larger rework of the theming within the next PR.

@ricoberger ricoberger force-pushed the rework-state-management branch 5 times, most recently from 533defc to 02c733e Compare January 4, 2023 21:27
Until now we used GetX for the state management in the app. Since we are
not very happy with this, we decided to remove GetX in all places and
use Provider for the state management. We also switched to the Navigator
instead of using GetX for the routing in the app. Last but not least we
also removed GetStorage and save the clusters, providers and app
settings via the Flutter Secure Storage package.

This commit also updates the required Dart and Go dependencies.

Note: The dark mode is not working in this version, because there will
be a larger rework of the theming within the next PR.
@ricoberger ricoberger marked this pull request as ready for review January 4, 2023 21:42
@ricoberger ricoberger merged commit 5416afe into main Jan 4, 2023
@ricoberger ricoberger deleted the rework-state-management branch January 4, 2023 21:42
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.

None yet

1 participant