wallet sign in driver #67

Merged
akinwale merged 2 commits from wallet-signin-driver into master 2019-10-29 13:36:12 +01:00
Showing only changes of commit 3bb8ee12eb - Show all commits

View file

@ -25,19 +25,20 @@ class WalletSignIn extends React.Component {
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
return (
<View style={walletStyle.signInContainer}>
<View style={walletStyle.signInSummaryRow}>
<Text style={walletStyle.signInTitle}>Sign in</Text>
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
<Text style={walletStyle.signInTitle}>Account Recommended</Text>
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
</View>
<View style={walletStyle.onboarding}>
<Text style={walletStyle.onboardingText}>
An account will allow you to earn rewards and keep your account and settings synced.{'\n\n'}
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
Without an account, you will not receive rewards, sync and backup services, or security updates.{'\n\n'}
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
An account with LBRY Inc. allows you to earn rewards, backup your wallet, and keep everything synced..
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
{'\n\n'}
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
Without an account, you assume all responsibility for securing your wallet and LBRY data.{'\n\n'}
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
</Text>
</View>
<View style={walletStyle.buttonRow}>
<Link style={walletStyle.continueLink} text={'Continue anyway'} onPress={this.onContinuePressed} />
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
<Button style={walletStyle.signInButton} theme={'light'} text={'Sign in'} onPress={this.onSignInPressed} />
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
<Link style={walletStyle.continueLink} text={'Skip Account'} onPress={this.onContinuePressed} />
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
<Button style={walletStyle.signInButton} theme={'light'} text={'Sign Up'} onPress={this.onSignInPressed} />
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
</View>
</View>
);

kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"
kauffj commented 2019-10-28 19:46:52 +01:00 (Migrated from github.com)
Review

"Sign In" in both the title and on the button, rather than "Sign in"

"Sign In" in both the title and on the button, rather than "Sign in"