-
Notifications
You must be signed in to change notification settings - Fork 0
feat(CrossAppLogin): Add UI #12
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
Conversation
|
Will review next week. |
397902d to
4d22a56
Compare
5ea9118 to
d3c0a40
Compare
4d22a56 to
0ee1b2e
Compare
|
I didn't add the screen header because it's not final on figma. @LunarX |
edcf427 to
e59bf90
Compare
LouisCAD
left a comment
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.
Looking good. A few things regarding todo comments.
app/src/main/kotlin/com/infomaniak/auth/manager/AccountsManager.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/com/infomaniak/auth/ui/screen/onboarding/OnboardingScreen.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/com/infomaniak/auth/ui/screen/onboarding/OnboardingScreen.kt
Outdated
Show resolved
Hide resolved
5f906ec to
c89e35c
Compare
c89e35c to
6eb48e4
Compare
app/src/main/kotlin/com/infomaniak/auth/ui/navigation/NavigationEntryProvider.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/com/infomaniak/auth/ui/navigation/NavigationEntryProvider.kt
Outdated
Show resolved
Hide resolved
app/src/main/kotlin/com/infomaniak/auth/ui/screen/onboarding/OnboardingScreen.kt
Outdated
Show resolved
Hide resolved
LunarX
left a comment
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.
I meant to select "request changes"
Add crossapplogin with getting account.
Keep lot of todo until login logic not ready