Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update some images #761

Merged
merged 3 commits into from Feb 23, 2015

Conversation

Projects
None yet
3 participants
Contributor

louisjc commented Feb 21, 2015

I've updated some images:

  • The chart and merchants map with more recent data
  • The resiliency image from weusecoin with the new version
Contributor

harding commented Feb 21, 2015

@louisjc thanks for the update! Also, thanks for taking the time to change the years embedded in all the translations.

For the price chart, I think we should keep your commit 5732a62 changing all the translations, but also add an extra commit changing the English translation to "Bitcoin price over time". (Maybe you also want to change the translation for any language you speak well to say the same thing.) Once that gets translated, we won't have to update the text again the next time we update the images. Thanks again!

(I'll check over the images more carefully when I get home to make sure they're compressed.)

Contributor

saivann commented Feb 21, 2015

I'd like to suggest the resiliency image should not be updated; it is not explicit what license applies to the new video, and it is likely CC BY-ND 4.0 (No deritative).

Contributor

louisjc commented Feb 21, 2015

@saivann Yep you're right, I had not thought that the license could have changed.
@harding The images where not compressed, I fixed it. If we change the text to be independent of the date, should we hotlink the image from bitcoincharts.com so it would be updated in real time ?

Contributor

harding commented Feb 22, 2015

@louisjc thanks for compressing the images!

I'd prefer to manually update the image every 6 or 12 months rather than hotlinking it. That seems like the polite thing to do, as well as ensuring the FAQ page displays correctly even if BitStamp is temporarily unavailable.

Thanks!

Contributor

louisjc commented Feb 22, 2015

@saivann I've removed the resiliency image commit.
@harding I've added the extra commit you suggested (in English and French).

@harding harding merged commit a811c9c into bitcoin-dot-org:master Feb 23, 2015

Contributor

harding commented Feb 23, 2015

@louisjc merged. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment