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

added learn more faq to wallet security section #1917

Merged
merged 5 commits into from Aug 29, 2018

Conversation

Projects
None yet
3 participants
@ykris45
Contributor

ykris45 commented Aug 28, 2018

unknown

@lbry-bot lbry-bot assigned seanyesmunt and unassigned seanyesmunt Aug 28, 2018

@ykris45

This comment has been minimized.

Show comment
Hide comment
@ykris45

ykris45 Aug 28, 2018

Contributor
Contributor

ykris45 commented Aug 28, 2018

@tzarebczan

This comment has been minimized.

Show comment
Hide comment
@tzarebczan

tzarebczan Aug 28, 2018

Member

@ykris45 how are you committing these changes, through the web interface? They should be submitted through git, so that formatting tools like prettier run on them. You can also run prettier locally in programs like Atom.

Member

tzarebczan commented Aug 28, 2018

@ykris45 how are you committing these changes, through the web interface? They should be submitted through git, so that formatting tools like prettier run on them. You can also run prettier locally in programs like Atom.

seanyesmunt added some commits Aug 29, 2018

@lbry-bot lbry-bot assigned seanyesmunt and unassigned seanyesmunt Aug 29, 2018

@seanyesmunt seanyesmunt merged commit ccd5f4c into lbryio:master Aug 29, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment