-
Notifications
You must be signed in to change notification settings - Fork 95
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
Move the core logic in LoginScreenCoordinator into the ViewModel. #3348
Conversation
Generated by 🚫 Danger Swift against ddb691b |
❌ 1 Tests Failed:
View the full list of 1 ❄️ flaky tests
To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard |
8db96a5
to
4ec3917
Compare
ElementX/Sources/Screens/Authentication/LoginScreen/LoginScreenViewModel.swift
Outdated
Show resolved
Hide resolved
ElementX/Sources/Screens/Authentication/LoginScreen/LoginScreenViewModel.swift
Show resolved
Hide resolved
ElementX/Sources/Screens/Authentication/LoginScreen/LoginScreenViewModel.swift
Show resolved
Hide resolved
ElementX/Sources/Screens/Authentication/LoginScreen/LoginScreenViewModel.swift
Show resolved
Hide resolved
ElementX/Sources/Screens/Authentication/LoginScreen/LoginScreenViewModelProtocol.swift
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just some nits
4ec3917
to
ddb691b
Compare
Quality Gate passedIssues Measures |
This PR does the same for the LoginScreenCoordinator as #3342 did for the ServerSelectionCoordinator, moving all the logic into the View Model to match the rest of the app.