Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Replaced Shadowcash with Monero #239

Merged
10 commits merged into from
Jun 29, 2017
Merged

Conversation

afighttilldeath
Copy link
Contributor

@afighttilldeath afighttilldeath commented Jun 29, 2017

Description

First time creating a pull request (or doing anything on GitHub) so please bear with me.

In response to issue #207, I have deleted Shadowcash from the cryptocurrency privacy listing and added Monero instead. I believe Monero has a proven track record for protecting people's privacy and would be a great addition to privacytools.io.

edit: changed HTML Preview to correctly redirect you.

HTML Preview

https://htmlpreview.github.io/?https://github.com/afighttilldeath/privacytools.io/blob/patch-1/index.html

@ghost ghost requested a review from kewde June 29, 2017 13:21
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Change image dimensions to 120x120, also please use an image of higher quality.

Proposed change: Switch Bitcoin and Monero due to Monero's focus on privacy.

@afighttilldeath
Copy link
Contributor Author

Okay, I agree. I'll switch Bitcoin and Monero and also add a higher quality image. Thanks!

@afighttilldeath
Copy link
Contributor Author

Okay, I switched Bitcoin and Monero sections. Also, added a better Monero icon.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thanks!

@ghost ghost merged commit d86c6bf into privacytools:master Jun 29, 2017
Copy link
Contributor

@kewde kewde left a comment

Choose a reason for hiding this comment

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

👍

@ghost ghost mentioned this pull request Jul 14, 2017
@kewde kewde mentioned this pull request Jul 14, 2017
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants