-
Notifications
You must be signed in to change notification settings - Fork 137
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
feat: add onfido onboarding for bank transfer #2699
Conversation
5fd0829
to
a0bd167
Compare
} | ||
|
||
return ( | ||
<Screen |
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.
It would be nice to lay out this screen similar to how we style other screens. Here is how its styled in the send flow. It uses padding of 20, and then the PrimaryButton is aligned with the bottom of the screen. https://github.com/GaloyMoney/galoy-mobile/blob/b60b1e086f3b3a4f47caf41f9a00dd010006bc28/app/screens/send-bitcoin-screen/send-bitcoin-details-screen.tsx#L643
keyboardOffset="navigationHeader" | ||
> | ||
<View style={styles.screenContainer}> | ||
<Text style={styles.textHeader}>{LL.FullOnboarding.requirements()}</Text> |
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.
Are you able to use a type="h1"
instead of a unique text style.
a0bd167
to
574c686
Compare
43b4f67
to
dd3792e
Compare
ef81688
to
6a714d5
Compare
6a714d5
to
9bc31fb
Compare
No description provided.