-
Notifications
You must be signed in to change notification settings - Fork 96
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
Implement onboarding - Closes #63 #572
Conversation
18db6ff
to
9ab72eb
Compare
a55540e
to
5e7d262
Compare
src/components/setting/setting.js
Outdated
@@ -83,6 +88,17 @@ class Setting extends React.Component { | |||
<p>{t('Lock ID’s automatically after 10 minutes.')}</p> | |||
</article> | |||
</div> | |||
<div> |
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.
this also needs this.showOnboardingSetting()
condition, because of swiping
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 made some wording suggestions.
src/components/onboarding/steps.js
Outdated
}, | ||
{ | ||
title: t('Timeout'), | ||
text: t('After 10 minutes of inactivity you will be logged out to prevent an unauthorized access to your Lisk ID. This will reset as soon as you become active again. You can also reset the timer anytime by clicking on the countdown.'), |
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.
After 10 minutes of not using your passphrase, your Lisk ID will be locked to prevent an unauthorized use of your Lisk ID. The timer resets when you send any transaction. You can also reset the timer by clicking "reset" when there are less than 5 minutes left.
src/components/onboarding/steps.js
Outdated
}, | ||
{ | ||
title: t('Lisk ID'), | ||
text: t('Click here to view your balance, Lisk ID and saved Lisk IDs.'), |
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.
Here you see you address and balance. You can click your account avatar to see all your saved IDs.
src/components/onboarding/steps.js
Outdated
}, | ||
{ | ||
title: t('Keep the overview'), | ||
text: t('Click here to go back to the main page of Lisk Hub and find all relevant statistics and ID information'), |
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.
Click here to go to the main page of Lisk Hub and find all relevant statistics and ID information.
src/components/onboarding/index.js
Outdated
run={this.props.isAuthenticated && isDesktop && (start || !this.isAlreadyOnboarded)} | ||
locale={{ | ||
last: (<span>{this.props.t('Complete')}</span>), | ||
skip: skip ? <span>{this.props.t('Use Lisk App')}</span> : <span>{this.props.t('Click here to skip')}</span>, |
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.
Use Lisk Hub
src/components/setting/setting.js
Outdated
}>{t('Start')}</button> | ||
<article> | ||
<h5>{t('Start the onboarding')}</h5> | ||
<p>{t('Take a quick tour to see how the Lisk App works.')}</p> |
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.
Lisk Hub
2c4026d
to
e1c4392
Compare
e1c4392
to
4c6d29a
Compare
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.
Thank you Gina
What was the problem?
We had no onboarding
How did I fix it?
By using the package
react-joyride
https://github.com/gilbarbara/react-joyride
How to test it?
This is WIP
Todo:
Tests will be done in #589
Mobile will be implemented in #588
What does it look like?
Review checklist