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

New "Spending Bitcoins" page (plus few other minor tweaks, wording fixes, etc.) #1584

Merged
merged 14 commits into from
Jun 25, 2017

Conversation

JustAHappyKid
Copy link

As per issue #1560.

Chris Wagner added 4 commits April 29, 2017 19:35
…mment about "speed-up instructions; plus make comment a link to relevant part of the documentation.
…ll links to UseBitcoins.info (except for the one on the Resources page, which expressly points to that site) with links to this new page.
… page to read "Find merchants and products".
…ple?" up-to-date given current realities (market-cap, sites that take BTC); plus minor wording tweaks.
products-online: "Find products for sale online"
products-online-text: "One common use for Bitcoin is making purchases online. There are hundreds of online shops and retailers that accept Bitcoin. Using a <a href=\"https://spendabit.co/\">search-engine like Spendabit</a> you can search through millions of products, all available for purchase with bitcoins."
directory: "Navigate a business directory"
directory-text: "You can also find many businesses <a href=\"/en/spend-bitcoin\">listed in online directories</a>. (<strong>Note:</strong> Unfortunately this directory has not been well-maintained and presently includes <em>many broken links</em> and references to defunct businesses.)"
Copy link
Contributor

@Mirobit Mirobit May 5, 2017

Choose a reason for hiding this comment

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

Usebitcoins.info/spendbitcoins.com instead of /en/spend-bitcoin

Copy link
Author

Choose a reason for hiding this comment

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

Good catch. Pushed a fix. :-)

@Mirobit
Copy link
Contributor

Mirobit commented May 5, 2017

Page looks good. Here is a screenshot:
spend

Has anyone checked spendbitcoins.com if they are more up-to-date than usebitcoin? We could use them instead.

Nit: Doesn't the spend-bitcoin.html belong into the _templates folder?

@JustAHappyKid
Copy link
Author

@microbit Regarding the _templates folder... I originally aimed to put the HTML there, but for whatever reason, it wouldn't seem to "register" with the Jekyll server, so trying to access the /en/spend-bitcoin URI would just give me back the homepage content (which it apparently does instead of giving a proper 404).

I grep'd through the source to try to determine where/if the other pages under _templates were "registered" (e.g., grep -R "protect-your-privacy" ./ | grep -v "./_translations/" | grep -v "./_site/"), to no avail. If you can give me a little guidance on that front, I'm happy to rework the patch to place the new page under _templates.

Regarding SpendBitcoins.com, I've not taken a close look at that site lately. (My hunch is it's little or no better -- I've never seen one of these directories be properly maintained.) I can poke around there, however, if you think it's worth a shot.

(FWIW, we [Spendabit] are planning to begin building a standard directory -- a la UseBitcoins.info/SpendBitcoins.com -- and maintain it, but that won't really help us here today; maybe 6 months down the road it will be an option.)

@wbnns
Copy link
Contributor

wbnns commented May 8, 2017

@JustAHappyKid Thanks a bunch for working on this. Could you generate a live preview of your work if you don't mind? If you don't know how, please let me know and I'll set one up.

@Mirobit
Copy link
Contributor

Mirobit commented May 11, 2017

@JustAHappyKid I have the same problem that jekyll doesn't want to generate the
spend-bitcoin.html file.

@wbnns wbnns added the On Hold label May 13, 2017
@kitsuphat
Copy link

Get feeds that have access to bitcoin wallet.

@JustAHappyKid
Copy link
Author

@wbnns I'll see if I can get a live preview up today. @microbit Should we just punt on moving it to the _templates directory for now?

…tcoin page, so it points to UseBitcoins.info (and not the Spending Bitcoin page itself).
@JustAHappyKid
Copy link
Author

@wbnns
Copy link
Contributor

wbnns commented May 16, 2017

@JustAHappyKid Ok, thanks! I'll review and will post comments later this week. There are a few issues, for example with the translation strings along with some other things. Good job setting up the preview!

Copy link
Contributor

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

@JustAHappyKid Great work on this. I just reviewed the PR and requested some changes. Also, we need to not make changes in any of the yml files that aren't en.yml because of the way translations work (they need to be updated first on Transifex, otherwise, your changes will get overwritten when translation strings get updated).

For now, you can just stick with the change to en.yml (and omit the ones to the other languages). Updating en.yml will ensure it gets up to Transifex so the other yml files get updated downstream.

If you have any questions about anything or if you need any help, please let me know - thanks again!

pagetitle: "Spending Bitcoin"
summary: "There are thousands of businesses, across the globe, that accept Bitcoin."
products-online: "Find products for sale online"
products-online-text: "One common use for Bitcoin is making purchases online. There are hundreds of online shops and retailers that accept Bitcoin. Using a <a href=\"https://spendabit.co/\">search-engine like Spendabit</a> you can search through millions of products, all available for purchase with bitcoins."
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove the hyphen from "search-engine" and make it two distinct words.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, please make the anchor text for the hyperlink just "Spendabit" instead of "search engine like Spendabit".

summary: "There are thousands of businesses, across the globe, that accept Bitcoin."
products-online: "Find products for sale online"
products-online-text: "One common use for Bitcoin is making purchases online. There are hundreds of online shops and retailers that accept Bitcoin. Using a <a href=\"https://spendabit.co/\">search-engine like Spendabit</a> you can search through millions of products, all available for purchase with bitcoins."
directory: "Navigate a business directory"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the word "to" may be missing from this (Navigate to a business directory).

Copy link
Author

Choose a reason for hiding this comment

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

I meant it to be more like "navigate through" (a la "navigate the ocean"). How about "Peruse a business directory"? Or I can just add the "to" -- either way.

Copy link
Contributor

Choose a reason for hiding this comment

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

@JustAHappyKid Ah, ok, nevermind, I see. :) Feel free to leave as-is.

products-online: "Find products for sale online"
products-online-text: "One common use for Bitcoin is making purchases online. There are hundreds of online shops and retailers that accept Bitcoin. Using a <a href=\"https://spendabit.co/\">search-engine like Spendabit</a> you can search through millions of products, all available for purchase with bitcoins."
directory: "Navigate a business directory"
directory-text: "You can also find many businesses <a href=\"http://usebitcoins.info/\">listed in online directories</a>. (<strong>Note:</strong> Unfortunately this directory has not been well-maintained and presently includes <em>many broken links</em> and references to defunct businesses.)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change the anchor text to just "online directories". Also, I'm not sure about the disclaimer, this sounds a bit negative and makes me wonder as a site visitor why we're linking to it in the first place (I realize though that there are issues with the directory, just thinking about if there's a better way we can word it).

Copy link
Author

Choose a reason for hiding this comment

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

Indeed. On the one hand it looks bad on Bitcoin.org's part to link to a site that's so poorly maintained; on the other hand we kinda need something there. Maybe if we just tone down the warning? E.g., "(Unfortunately this directory includes a number of links to defunct businesses and old websites, at present.)"

directory: "Navigate a business directory"
directory-text: "You can also find many businesses <a href=\"http://usebitcoins.info/\">listed in online directories</a>. (<strong>Note:</strong> Unfortunately this directory has not been well-maintained and presently includes <em>many broken links</em> and references to defunct businesses.)"
business-map: "Find local businesses"
business-map-text: "There are also many local businesses, like cafes and restaurants, that accept Bitcoin. You can <a href=\"https://coinmap.org/\">use Coinmap.org</a> to browse thousands of businesses across the globe."
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change the anchor text to just "Coinmap.org" (instead of "use Coinmap.org").

@@ -0,0 +1,20 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

This file should be set up and placed in the _templates instead of /en/.

Copy link
Author

@JustAHappyKid JustAHappyKid May 22, 2017

Choose a reason for hiding this comment

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

This is the only request I'm not exactly sure how to handle... As I mentioned in a comment above, I was not able to get Jekyll to respond to files (to compile, generate, etc.) I placed under _templates -- which was where I initially attempted to place the new page. @microbit seemed to have the same issue. Any thoughts/hints?

Copy link
Contributor

Choose a reason for hiding this comment

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

@JustAHappyKid At first glance it looks like you didn't add the url at the bottom of en.yml.

@@ -350,7 +350,7 @@
howitworkstxt1: "From a user perspective, Bitcoin is nothing more than a mobile app or computer program that provides a personal Bitcoin wallet and allows a user to send and receive bitcoins with them. This is how Bitcoin works for most users."
howitworkstxt2: "Behind the scenes, the Bitcoin network is sharing a public ledger called the \"block chain\". This ledger contains every transaction ever processed, allowing a user's computer to verify the validity of each transaction. The authenticity of each transaction is protected by digital signatures corresponding to the sending addresses, allowing all users to have full control over sending bitcoins from their own Bitcoin addresses. In addition, anyone can process transactions using the computing power of specialized hardware and earn a reward in bitcoins for this service. This is often called \"mining\". To learn more about Bitcoin, you can consult the <a href=\"#how-it-works#\">dedicated page</a> and the <a href=\"#bitcoin-paper#\">original paper</a>."
used: "Is Bitcoin really used by people?"
usedtxt1: "Yes. There is a <a href=\"http://usebitcoins.info/\">growing number of businesses</a> and individuals using Bitcoin. This includes brick and mortar businesses like restaurants, apartments, law firms, and popular online services such as Namecheap, WordPress, and Reddit. While Bitcoin remains a relatively new phenomenon, it is growing fast. At the end of August 2013, the <a href=\"https://bitcoincharts.com/bitcoin/\">value of all bitcoins in circulation</a> exceeded US$ 1.5 billion with millions of dollars worth of bitcoins exchanged daily."
usedtxt1: "Yes. There is a <a href=\"/en/spend-bitcoin\">growing number of businesses</a> and individuals using Bitcoin. This includes brick-and-mortar businesses like restaurants, apartments, and law firms, as well as popular online services such as Namecheap, Overstock.com, and Reddit. While Bitcoin remains a relatively new phenomenon, it is growing fast. At the end of April 2017, the <a href=\"https://bitcoincharts.com/bitcoin/\">total value of all existing bitcoins</a> exceeded 20-billion US dollars, with millions of dollars worth of bitcoins exchanged daily."
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change "Yes, There is a" to "Yes, There are a" (I know it was already like that)

Also, please change "20-billion" to "20 billion" (no hyphen)

Copy link
Author

Choose a reason for hiding this comment

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

Wasn't sure whether the verb (is/are) should be acting on "number" (singular) or "businesses" (plural), so just left it as it was previously... But I concur, "There are" reads easier. :-)

Copy link
Contributor

Choose a reason for hiding this comment

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

@JustAHappyKid Ok, cool. 👍

@JustAHappyKid
Copy link
Author

Okay, sounds like I just need to revert all changes under _translations except those to en.yml, plus make the specific wording/markup tweaks outlined. Will try to tackle it this week, and I've got a few comments I'll add below your specific notes...

# http://opensource.org/licenses/MIT.

layout: base
lang: en
Copy link
Contributor

@wbnns wbnns May 26, 2017

Choose a reason for hiding this comment

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

@JustAHappyKid Please remove line 6: lang: en

Chris Wagner added 6 commits May 31, 2017 13:27
…n files that relate to new "Spend Bitcoin" page.
…ving hyphen), and for link to Spendabit.co only hyperlink the site name ("Spendabit") as opposed to the whole phrase "search engine like Spendabit".
… page) to just wrap text "online directories".
…unct businesses/websites at UseBitcoins.info (on new Spend page).
… to only wrap text "Coinmap.org" (not "use Coinmap.org").
…'spend-bitcoin' to 'url' section of en.yml.
@JustAHappyKid
Copy link
Author

@wbnns Okay, I've moved spend-bitcoin.html to _templates and made the requested wording/markup tweaks -- hopefully I didn't miss anything!

@Mirobit
Copy link
Contributor

Mirobit commented Jun 15, 2017

LGTM

spend

Copy link
Contributor

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

@JustAHappyKid Thanks for making the additional changes. We're almost there and everything you've done looks great. After further review of the Use Bitcoins website, let's go ahead and remove that section. I think sending people to that site may do more harm than good in its current condition.

products-online: "Find products for sale online"
products-online-text: "One common use for Bitcoin is making purchases online. There are hundreds of online shops and retailers that accept Bitcoin. Using a search engine like <a href=\"https://spendabit.co/\">Spendabit</a> you can search through millions of products, all available for purchase with bitcoins."
directory: "Navigate a business directory"
directory-text: "You can also find many businesses listed in <a href=\"http://usebitcoins.info/\">online directories</a>. (Note: Unfortunately this directory includes a number of defunct businesses and websites.)"
Copy link
Contributor

Choose a reason for hiding this comment

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

After additional review, let's just remove the "Navigate a business directory" section. The usebitcoins.info site is heavily ad-laden (several are somewhat scammy looking) and as several others have mentioned, there are a significant number of broken links.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can replace it with the 99 bitcoin List :https://99bitcoins.com/who-accepts-bitcoins-payment-companies-stores-take-bitcoins/
It is not a directory but a List with the mayor merchants.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Mirobit Thanks for the suggestion. We could alternatively do this, too. 👍

@JustAHappyKid It's your call!

<h2 id="products-online"><span class="fa fa-shopping-cart fa-lg"></span> {% translate products-online %}</h2>
<p>{% translate products-online-text %}</p>

<h2 id="directory"><span class="fa fa-book fa-lg"></span> {% translate directory %}</h2>
Copy link
Contributor

Choose a reason for hiding this comment

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

As per my other comment, let's remove this section, along with the strings in the other file.

Chris Wagner added 2 commits June 21, 2017 13:33
…ns.com page (that includes links to many BTC-accepting merchants), and remove disclaimer (that was aimed at UseBitcoins.info) about directory containing defunct businesses.
@JustAHappyKid
Copy link
Author

@wbnns @Mirobit Okay, hopefully we're good to go... I opted to retain the "directory" section, using the 99bitcoins.com link (anticipating that we will be able to replace that with a link to a legit directory in the near future). I also removed UseBitcoins.info from the Resources page.

@wbnns
Copy link
Contributor

wbnns commented Jun 24, 2017

@JustAHappyKid Awesome, this looks great.

Unless others object, this will be merged on Sunday, June 25th.

@wbnns wbnns merged commit 175654a into bitcoin-dot-org:master Jun 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants