Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

fix typos and grammar #4880

Merged
merged 1 commit into from
Mar 12, 2017
Merged

fix typos and grammar #4880

merged 1 commit into from
Mar 12, 2017

Conversation

JohnDoe389
Copy link
Contributor

fix typos and grammar

btw, i love Parity...can i have a job?

@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. M7-ui labels Mar 12, 2017
@gavofyork gavofyork merged commit 18b2983 into openethereum:master Mar 12, 2017
@gavofyork
Copy link
Contributor

sure - send a cv to info@parity.io

overview_2: `Are you a developer? Add some features to manage contracts are interact with application deployments.`,
overview_3: `Are you a miner or run a large-scale node? Add the features to give you all the information needed to watch the node operation.`,
overview_0: `Manage the available application views, using only the parts of the application applicable to you.`,
overview_1: `Are you an end-user? The defaults are setup for both beginner and advanced users alike.`,
Copy link
Contributor

Choose a reason for hiding this comment

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

To nitpick further, should be "set up"

Copy link
Contributor

Choose a reason for hiding this comment

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

actually, this wasn't wrong before: a "setup" is a real noun, so "the defaults are setups (that ...)" is fine

/>
</div>
<div>
<FormattedMessage
id='settings.views.overview_1'
defaultMessage='Are you an end-user? The defaults are setups for both beginner and advanced users alike.'
defaultMessage='Are you an end-user? The defaults are setup for both beginner and advanced users alike.'
Copy link
Contributor

Choose a reason for hiding this comment

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

same comments apply here

@JohnDoe389
Copy link
Contributor Author

JohnDoe389 commented Mar 12, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants