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

Remove onboarding - Closes #1870 #1937

Merged
merged 1 commit into from
Apr 17, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 0 additions & 27 deletions i18n/locales/en/common.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
"64 bytes left": "64 bytes left",
"About": "About",
"Academy": "Academy",
"Access extra features": "Access extra features",
"Account": "Account",
"Account Balance": "Account Balance",
"Account Info": "Account Info",
Expand All @@ -36,7 +35,6 @@
"Additional fee: {{fee}} LSK": "Additional fee: {{fee}} LSK",
"Address": "Address",
"Advanced features": "Advanced features",
"After 10 minutes of not using your passphrase,\n // your Lisk ID will be locked to prevent an unauthorized use of your Lisk ID. .\n // The timer will reset as soon as you make an transaction.\n // After 5 minutes, you can also reset the timer by clicking on \"reset\".": "After 10 minutes of not using your passphrase,\n // your Lisk ID will be locked to prevent an unauthorized use of your Lisk ID. .\n // The timer will reset as soon as you make an transaction.\n // After 5 minutes, you can also reset the timer by clicking on \"reset\".",
"After logging out of your account you will be able to access the Dashboard, Settings and Search.": "After logging out of your account you will be able to access the Dashboard, Settings and Search.",
"After registration, your second passphrase will be required when confirming every transaction and every vote.": "After registration, your second passphrase will be required when confirming every transaction and every vote.",
"All": "All",
Expand Down Expand Up @@ -72,7 +70,6 @@
"By voting you decide who is trusted to verify transactions and maintain the Lisk network, whilst collecting the rewards for doing so.": "By voting you decide who is trusted to verify transactions and maintain the Lisk network, whilst collecting the rewards for doing so.",
"Bytes counter": "Bytes counter",
"Cancel": "Cancel",
"Change account settings, enable delegate voting.": "Change account settings, enable delegate voting.",
"Check for updates...": "Check for updates...",
"Check spelling – name does not exist on mainnet": "Check spelling – name does not exist on mainnet",
"Choose": "Choose",
Expand All @@ -81,12 +78,8 @@
"Choose your Avatar": "Choose your Avatar",
"Choose your name": "Choose your name",
"Clear All filters": "Clear All filters",
"Click here to go to the main page of Lisk Hub and find all relevant statistics and ID information": "Click here to go to the main page of Lisk Hub and find all relevant statistics and ID information",
"Click here to skip": "Click here to skip",
"Click here to transfer funds between Lisk IDs and access your transaction history.": "Click here to transfer funds between Lisk IDs and access your transaction history.",
"Close": "Close",
"Coming soon.": "Coming soon.",
"Complete": "Complete",
"Confirm": "Confirm",
"Confirm (Fee: 1 LSK)": "Confirm (Fee: 1 LSK)",
"Confirm (Fee: {{fee}} LSK)": "Confirm (Fee: {{fee}} LSK)",
Expand Down Expand Up @@ -141,7 +134,6 @@
"Delegate statistics": "Delegate statistics",
"Delegate stats": "Delegate stats",
"Delegate vote_noun": "Delegate vote",
"Delegate voting": "Delegate voting",
"Delegates": "Delegates",
"Delegates have great responsibility within the Lisk system for securing the blockchain. Becoming a delegate requires the registration of a name. The top 101 delegates are eligible to forge.": "Delegates have great responsibility within the Lisk system for securing the blockchain. Becoming a delegate requires the registration of a name. The top 101 delegates are eligible to forge.",
"Details": "Details",
Expand All @@ -168,7 +160,6 @@
"Every transaction needs to be confirmed and forged into Lisks blockchain network. \n Such operations require hardware resources and because of that there is a small fee for processing those.": "Every transaction needs to be confirmed and forged into Lisks blockchain network. \n Such operations require hardware resources and because of that there is a small fee for processing those.",
"Explain Blockchain Like I'm 5": "Explain Blockchain Like I'm 5",
"Explore as a Guest": "Explore as a Guest",
"Explore the network": "Explore the network",
"Failed to connect to node": "Failed to connect to node",
"Failed to connect: Node {{address}} is not active": "Failed to connect: Node {{address}} is not active",
"Fee": "Fee",
Expand All @@ -189,7 +180,6 @@
"Give feedback about this feature": "Give feedback about this feature",
"Go Back": "Go Back",
"Go back to Dashboard": "Go back to Dashboard",
"Go to \"Help\" on your bottom bar to start the onboarding whenever you need.": "Go to \"Help\" on your bottom bar to start the onboarding whenever you need.",
"Go to Confirmation": "Go to Confirmation",
"Go to settings": "Go to settings",
"Go to your Dashboard": "Go to your Dashboard",
Expand All @@ -201,7 +191,6 @@
"Height": "Height",
"Help": "Help",
"Help Center": "Help Center",
"Here you see your address, balance, and avatar.": "Here you see your address, balance, and avatar.",
"Hide passphrase": "Hide passphrase",
"Hide the QR code": "Hide the QR code",
"How do you like the hub?": "How do you like the hub?",
Expand Down Expand Up @@ -232,7 +221,6 @@
"It is highly recommended to delete the PDF and remove it from your trash after printing.": "It is highly recommended to delete the PDF and remove it from your trash after printing.",
"It is recommended that you initialize your Lisk ID.": "It is recommended that you initialize your Lisk ID.",
"It will cost you only the usual {{fee}} LSK transaction fee.": "It will cost you only the usual {{fee}} LSK transaction fee.",
"Keep the overview": "Keep the overview",
"Keep this safe, never throw it away or share it with anyone.": "Keep this safe, never throw it away or share it with anyone.",
"Keep track of any Lisk ID balance. Only you will see who you bookmarked.": "Keep track of any Lisk ID balance. Only you will see who you bookmarked.",
"LISK Hub counts your message by bytes so keep in mind \n that the length on your message may vary in different languages. \n Different characters may consume different amount of bytes space.": "LISK Hub counts your message by bytes so keep in mind \n that the length on your message may vary in different languages. \n Different characters may consume different amount of bytes space.",
Expand Down Expand Up @@ -271,7 +259,6 @@
"Look at your Ledger for confirmation": "Look at your Ledger for confirmation",
"Look at your Ledger for confirmation of second signature": "Look at your Ledger for confirmation of second signature",
"Mainnet": "Mainnet",
"Manage your application": "Manage your application",
"Market": "Market",
"Max": "Max",
"Max 20 characters, a-z and 0-1, no special characters except for “!@$&.”": "Max 20 characters, a-z and 0-1, no special characters except for “!@$&.”",
Expand All @@ -293,7 +280,6 @@
"Name is available": "Name is available",
"Name too long": "Name too long",
"Names are unique. Once you register a name, it can't be changed.": "Names are unique. Once you register a name, it can't be changed.",
"New to Hub? Take a tour": "New to Hub? Take a tour",
"New to Lisk? ": "New to Lisk? ",
"New version available": "New version available",
"News": "News",
Expand All @@ -313,7 +299,6 @@
"Ok": "Ok",
"Okay": "Okay",
"On your dashboard": "On your dashboard",
"Onboarding whenever you need": "Onboarding whenever you need",
"Oops! Wrong passphrase": "Oops! Wrong passphrase",
"Oops, looks like something went wrong. Please try again.": "Oops, looks like something went wrong. Please try again.",
"Open the Lisk App": "Open the Lisk App",
Expand Down Expand Up @@ -359,7 +344,6 @@
"Reference": "Reference",
"Reference (optional)": "Reference (optional)",
"Register": "Register",
"Register and oversee your decentralized application here. Available soon.": "Register and oversee your decentralized application here. Available soon.",
"Registered": "Registered",
"Registrant": "Registrant",
"Release Notes": "Release Notes",
Expand Down Expand Up @@ -387,7 +371,6 @@
"Search for a delegate": "Search for a delegate",
"Search for answers in our extensive ": "Search for answers in our extensive ",
"Search for delegate, Lisk ID, transaction ID": "Search for delegate, Lisk ID, transaction ID",
"Search for delegates, addresses and transactions.": "Search for delegates, addresses and transactions.",
"Second passphrase": "Second passphrase",
"Second passphrase (Fee: 5 LSK)": "Second passphrase (Fee: 5 LSK)",
"Second passphrase registration": "Second passphrase registration",
Expand Down Expand Up @@ -434,7 +417,6 @@
"Sorry": "Sorry",
"Standby": "Standby",
"Start here": "Start here",
"Start the tour": "Start the tour",
"Stats": "Stats",
"Status": "Status",
"Step 1 / 4": "Step 1 / 4",
Expand All @@ -446,7 +428,6 @@
"Success!": "Success!",
"Sum of LSK of all accounts who voted for a delegate": "Sum of LSK of all accounts who voted for a delegate",
"Switch networks (Main-/Testnet, Custom)": "Switch networks (Main-/Testnet, Custom)",
"Take a tour to see how it works. It should not take longer than 5 minutes": "Take a tour to see how it works. It should not take longer than 5 minutes",
"Tell us what you think": "Tell us what you think",
"Terms of Use": "Terms of Use",
"Testnet": "Testnet",
Expand All @@ -466,7 +447,6 @@
"This is your Lisk ID": "This is your Lisk ID",
"This process should take only 10 seconds but may take up to 15 minutes.": "This process should take only 10 seconds but may take up to 15 minutes.",
"This signed message can prove that you are the owner of the account, since only your passphrase can produce it.": "This signed message can prove that you are the owner of the account, since only your passphrase can produce it.",
"Timeout": "Timeout",
"Timeout soon": "Timeout soon",
"Title": "Title",
"Title too long": "Title too long",
Expand Down Expand Up @@ -505,13 +485,11 @@
"Updates downloaded, application has to be restarted to apply the updates.": "Updates downloaded, application has to be restarted to apply the updates.",
"Uptime": "Uptime",
"Upvotes": "Upvotes",
"Use Lisk Hub": "Use Lisk Hub",
"Use extensions from https://raw.githubusercontent.com/michaeltomasik/extensions-lisk/master/": "Use extensions from https://raw.githubusercontent.com/michaeltomasik/extensions-lisk/master/",
"Use the sharing link to easily request any amount of LSK from Lisk Hub or Lisk Mobile users.": "Use the sharing link to easily request any amount of LSK from Lisk Hub or Lisk Mobile users.",
"Username": "Username",
"Version": "Version",
"View": "View",
"View forging delegates and vote for the ones you support.": "View forging delegates and vote for the ones you support.",
"Visit Help Center": "Visit Help Center",
"Visit Lisk.Chat": "Visit Lisk.Chat",
"Vote Transaction": "Vote Transaction",
Expand All @@ -529,7 +507,6 @@
"We recommend including date & time or a specific keyword.": "We recommend including date & time or a specific keyword.",
"We strongly recommend to store your passphrase in a safe place, such as on a password manager or paperwallet.": "We strongly recommend to store your passphrase in a safe place, such as on a password manager or paperwallet.",
"Welcome back": "Welcome back",
"Welcome to Lisk Hub": "Welcome to Lisk Hub",
"Welcome to the Lisk Hub!": "Welcome to the Lisk Hub!",
"What is Lisk Academy?": "What is Lisk Academy?",
"What is Lisk ID?": "What is Lisk ID?",
Expand All @@ -548,9 +525,6 @@
"You can explore Lisk network using Hub without signing in.": "You can explore Lisk network using Hub without signing in.",
"You can find the LSK token on all of the worlds top exchanges and send them to your unique Lisk address:": "You can find the LSK token on all of the worlds top exchanges and send them to your unique Lisk address:",
"You can now securely manage your LSK tokens.": "You can now securely manage your LSK tokens.",
"You can now use Lisk Hub.<br": {
" If you want to repeat the onboarding, navigate to \"Help\" on the sidebar.": "You can now use Lisk Hub.<br> If you want to repeat the onboarding, navigate to \"Help\" on the sidebar."
},
"You can print your passphrase to be stored in a safe place. ": "You can print your passphrase to be stored in a safe place. ",
"You can share your Lisk ID with anyone you wish, but never reveal your passphrase to anyone as it would allow full access to your account.": "You can share your Lisk ID with anyone you wish, but never reveal your passphrase to anyone as it would allow full access to your account.",
"You can use tab or space to go to the next field.": "You can use tab or space to go to the next field.",
Expand Down Expand Up @@ -582,7 +556,6 @@
"Your session was timed out after 10 minutes of no network activity. You may continue to use certain sections of your Lisk Hub or sign back in to access everything.": "Your session was timed out after 10 minutes of no network activity. You may continue to use certain sections of your Lisk Hub or sign back in to access everything.",
"Your signed message": "Your signed message",
"Your votes are being processed. It may take up to 10 minutes for it to be secured in the blockchain.": "Your votes are being processed. It may take up to 10 minutes for it to be secured in the blockchain.",
"You’ve completed the tour!": "You’ve completed the tour!",
"Zero not allowed": "Zero not allowed",
"[Today], HH:mm": "[Today], HH:mm",
"[Tomorrow], HH:mm": "[Tomorrow], HH:mm",
Expand Down
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,6 @@
"react-css-themr": "=2.1.2",
"react-dom": "16.2.1",
"react-i18next": "=6.0.6",
"react-joyride": "1.11.4",
"react-piwik": "1.6.0",
"react-redux": "5.0.6",
"react-router": "4.2.0",
Expand Down
1 change: 0 additions & 1 deletion src/components/app/global.css
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
@import './variables.css';
@import '../onboarding/onboarding.css';

/*****************************************

Expand Down
4 changes: 0 additions & 4 deletions src/components/app/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ import StatusBar from '../statusBar';

import routes from '../../constants/routes';
// eslint-disable-next-line import/no-named-as-default
import Onboarding from '../onboarding';

class App extends React.Component {
constructor() {
Expand Down Expand Up @@ -49,7 +48,6 @@ class App extends React.Component {

return (
<OfflineWrapper>
<Onboarding appLoaded={this.state.loaded} />
<Dialog />
{
routesV2Layout.filter(route => route.path === location.pathname).length > 0
Expand All @@ -58,7 +56,6 @@ class App extends React.Component {
`${stylesV2.v2Wrapper} ${stylesV2.loaded} appLoaded` :
`${styles.v2Wrapper}`
} ref={(el) => { this.main = el; }}>
<div id='onboardingAnchor'></div>
<Switch>
{this.state.loaded &&
routesV2Layout.map((route, key) => (
Expand All @@ -81,7 +78,6 @@ class App extends React.Component {
<TopBar />
<section>
<div className={styles.mainBox}>
<div id='onboardingAnchor'></div>
<Switch>
{this.state.loaded &&
<Route path={routes.explorer.path} component={() => (
Expand Down
11 changes: 0 additions & 11 deletions src/components/help/help.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,6 @@ import Piwik from '../../utils/piwik';

/* eslint-disable class-methods-use-this */
class Help extends React.Component {
initOnboarding() {
Piwik.trackingEvent('Help', 'button', 'Init onboarding');
this.props.settingsUpdated({ onBoarding: true });
}

visitHelpCenter() {
const win = window.open(links.helpCenter, '_blank');
win.focus();
Expand All @@ -32,12 +27,6 @@ class Help extends React.Component {
<header>
<h2>{this.props.t('Help')}</h2>
<p>{this.props.t('Browse through our tutorials, check out the FAQ or connect with our knowledgeable community.')}</p>
{
this.props.account.address ?
<a className='help-onboarding' onClick={() => this.initOnboarding() }>{this.props.t('New to Hub? Take a tour')}
<FontIcon>arrow-right</FontIcon>
</a> : null
}

<a
className={'terms-of-use'}
Expand Down
11 changes: 0 additions & 11 deletions src/components/help/help.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,18 +31,7 @@ describe('Help Page', () => {
it('renders three help article sections with proper btn callbacks', () => {
expect(wrapper).to.have.exactly(3).descendants('.help-articles article');

wrapper.find('.help-onboarding').simulate('click');
expect(props.settingsUpdated).to.have.been.calledWith({ onBoarding: true });

wrapper.find('.help-visit-center').simulate('click');
expect(windowOpenMock).to.have.been.calledWith(links.helpCenter, '_blank');

const propsNotLoggedIn = {
...props,
};
delete propsNotLoggedIn.account.address;
wrapper.setProps(propsNotLoggedIn);
wrapper.update();
expect(wrapper).to.have.exactly(0).descendants('.help-onboarding');
});
});
Loading