MMM-cryptocurrency - v1.4

[matteodanelli]

@SpecialRob Yes weird. But thank you for reporting it.
I’ll investigate.

[Klizzy]

@matteodanelli nice work!
just in time for my recent changes ☺️
(you get the pull request in a couple of minutes).

I created a second display option with less information overall, but with icons.
I had to do this, because i have many modules on the Mirror and therefor the readability overall decreased with so many informations.

But now we can simply choose between more details or the “compact” version.
Here is a little Sneakpeak, until i finish the changes of the Readme accordingly to my changes:

[fox]

Very nice !
Thank you for doing this.

[olexs]

Hey,

thanks for this module! Exactly what I wanted for my mirror.

I took the liberty of implementing two new features, pull requests are up on Github:

1. Show weekly price graphs from coinmarketcap: https://github.com/matteodanelli/MMM-cryptocurrency/pull/6
   

2. Round the prices to a total number of significant digits, to keep a higher resolution on low-value prices: https://github.com/matteodanelli/MMM-cryptocurrency/pull/7
   
   The property is set to 5 in the example. Note how BTC and ETH are both still showing two decimals, but XRP is showing 4, for a total of 5 or more digits.

Any comments are welcome! I tried to keep your coding conventions intact, though it wasn’t much code that I added :)

[matteodanelli]

@olexs Amazing work!
Thanks for pull requests! Already merged :)
Thank you so much!

[fox]

i love the module. thank you !

But i dont get logo working. it stays always at “detail” and i also enabled graph, but they are not shown.
any existing bug? thx

[olexs]

@fox Can you post your module configuration from config.js? And have you pulled the latest code (master branch from https://github.com/matteodanelli/MMM-cryptocurrency)?

[Klizzy]

@fox please get the latest release of this Module from Github and replace the folder on your Mirror with the new one. Because you have to Update it manually if you want the graph or newer features.
If you use git, you can just do “git pull origin master” in the console.

@olexs thank you for your awesome work!

I tried for some time to implement the graph but i couldn’t find a solution…

@matteodanelli
I added some features and submitted an pull request.

• Colored logos (optional)
• additional logo support for Nem and Stratis
• Graph Support for Stratis
• now gets the top25 results of coinmareket

It would be nice if the currencies are sorted by total market volume (like on coin market). At the moment its ordered by the definitions in the config section.
Maybe someone has more time as i in the next days?   ☺️

And what do you guys think if we start to add release notes / change log to the readme?

[s199]

Hello
thank you for this addon :) it works fine and looks nice.
Is it possible to add an option to set a certain amount for each coin ? Some coin are really low in value like Dogecoin and the price for ONE single coin is always on 0. (ex 1000 dogecoin so at least its worth something :)
Also is it possible to use BTC as main currency, so everything is converted in BTC insted of USD or EUR ?
thanks a lot
++

edit : now that i think about it, ideally there should be the price BTC > USD (or EURO etc) and then all the other crypto u want converted in BTC. that would make more sens as BTC is the only way to trade these alts coin. two different coin convertion then :)

[matteodanelli]

@s199 At the moment, service which provides data is not able to convert in BTC, but only to a specified set of currencies, none virtual.
For the Dogecoin request. It could make sense but it requires a bit of configuration and so I need some time to develop it. But I’ll do.

For what I know, you can trade every crypto currency in EUR or USD or any other, not only BTC.