Skip to content
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

Merged
merged 24 commits into from
Mar 28, 2018
Merged

Implement onboarding - Closes #63 #572

merged 24 commits into from
Mar 28, 2018

Conversation

ginacontrino
Copy link
Contributor

@ginacontrino ginacontrino commented Mar 19, 2018

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:

  • Timer step can be conditional as delegates step: @pending, waiting for Taiko for a solution
  • Start on-boarding from menu doesn’t work after finishing the automatic one, only after page refresh

Tests will be done in #589
Mobile will be implemented in #588

What does it look like?

screen shot 2018-03-19 at 16 15 21

screen shot 2018-03-20 at 11 38 42

Review checklist

  • The PR solves Implement on-boarding process #63
  • All new code is covered with unit tests
  • All new features are covered with e2e tests
  • All new code follows best practices

@ginacontrino ginacontrino self-assigned this Mar 19, 2018
@ginacontrino ginacontrino force-pushed the 63-onboarding branch 5 times, most recently from 18db6ff to 9ab72eb Compare March 20, 2018 15:45
@ginacontrino ginacontrino force-pushed the 63-onboarding branch 7 times, most recently from a55540e to 5e7d262 Compare March 23, 2018 09:16
@@ -83,6 +88,17 @@ class Setting extends React.Component {
<p>{t('Lock ID’s automatically after 10 minutes.')}</p>
</article>
</div>
<div>
Copy link
Contributor

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

Copy link
Contributor

@slaweet slaweet left a 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.

},
{
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.'),
Copy link
Contributor

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.

},
{
title: t('Lisk ID'),
text: t('Click here to view your balance, Lisk ID and saved Lisk IDs.'),
Copy link
Contributor

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.

},
{
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'),
Copy link
Contributor

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.

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>,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use Lisk Hub

}>{t('Start')}</button>
<article>
<h5>{t('Start the onboarding')}</h5>
<p>{t('Take a quick tour to see how the Lisk App works.')}</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lisk Hub

@ginacontrino ginacontrino force-pushed the 63-onboarding branch 3 times, most recently from 2c4026d to e1c4392 Compare March 27, 2018 09:28
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Gina

@ginacontrino ginacontrino merged commit e71449c into 0.4.0 Mar 28, 2018
@ginacontrino ginacontrino deleted the 63-onboarding branch March 28, 2018 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants