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.

Sign up for GitHub

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(config): move business logic to ViewModel #733

Merged

andrekir merged 1 commit into master from config_cleanup

Sep 17, 2023

Member

andrekir commented Sep 16, 2023

Refactor UI to simply consume data from ViewModel improving code organization and separation of concerns:

generalized ResponseState for wider use;
consolidate state variables into RadioConfigState;
moved config getters/setters processing to ViewModel;
added error handling for a few extra cases.


refactor(config): move business logic to ViewModel

d13ca20

andrekir merged commit 7869243 into master

1 check passed

andrekir deleted the config_cleanup branch

September 17, 2023 09:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment