EvanRespaut / Equate

Unit converting calculator
GNU General Public License v3.0
67 stars 17 forks source link

Cryptocurrency rates not working #33

Closed ghost closed 4 years ago

ghost commented 6 years ago

For a few days now, the cryptocurrency rates haven't been working. Any ideas why?

Actually, looking at it, I think all the rates are down.

Thanks

EvanRespaut commented 6 years ago

Looks like the yahoo source I've been pulling from is down: https://finance.yahoo.com/webservice/v1/symbols/allcurrencies/quote

Not sure if it's down for good or just temporary.

ghost commented 6 years ago

You can get the ECB rates directly. I believe it's updated once a day at 3pm: http://www.ecb.europa.eu/stats/eurofxref/eurofxref-daily.xml

Not sure how many currencies you get from yahoo, but a quick look at the above feed shows 32 currencies against the euro. Obviously from there you can calculate any pair.

As for cryptocurrencies, I posted it elsewhere but to save jumping around, here's the top 1000: https://api.coinmarketcap.com/v1/ticker/?limit=1000

I suppose grabbing info from multiple sources would help in situations like this.

tmlee commented 6 years ago

For cryptocurrency data, I would like to suggest giving our API a shot at https://www.coingecko.com/api/docs/v3

Should be straighforward to get started and easy to consumer.

Let me know your thoughts

EvanRespaut commented 6 years ago

That ECB api looks pretty solid, although not nearly the same number as Yahoo. That had something like all 160 currencies plus precious metals. If I switch to ECB, I'll should probably delete the other currencies that will always be outdated.

For crypto, I'm already using coinmarketcap, although the broken Yahoo call breaks that too, so I can fix that.

tmlee commented 6 years ago

Sounds good. in any case cmc will soon require api key and registration, if you are looking for a workaround that you may consider our api as an option

cbix commented 6 years ago

Any decisions so far on changing the source of the exchange rates? You could also allow the user to choose from different sources, like this app does: https://github.com/hwki/SimpleBitcoinWidget However that's not trivial to implement probably...

EvanRespaut commented 6 years ago

Added support to pull rates from ECB. Update is live as of yesterday. Let me know if there are issues.

ghost commented 6 years ago

It would nice if a lot more cryptos were included, or at least the top 100.

Gold and silver is no longer available?

Thank you very much.

ghost commented 6 years ago

Tip incoming

cbix commented 6 years ago

Added support to pull rates from ECB. Update is live as of yesterday. Let me know if there are issues.

I was not able to build the latest master, will try again later. If it's the final 1.7 release, could you tag it and maybe upload the APK to the github releases (or attach here as .zip) until it appears on F-Droid?

EDIT: nevermind, I updated Android Studio and it worked. But now UAH (ukrainian hrywnia) is missing :(

cbix commented 6 years ago

https://f-droid.org/wiki/page/com.llamacorp.equate/lastbuild

F-droid seems to have issues buidling 1.7, but it looks like a problem in the f-droid tools:

FAILURE: Build failed with an exception.
* What went wrong:
A problem occurred configuring root project 'com.llamacorp.equate'.
> Could not resolve all files for configuration ':classpath'.
    > Could not download gradle-core.jar (com.android.tools.build:gradle-core:2.3.3)
        > Could not get resource 'https://jcenter.bintray.com/com/android/tools/build/gradle-core/2.3.3/gradle-core-2.3.3.jar'.
            > Connection reset
* Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
BUILD FAILED
EvanRespaut commented 5 years ago

I made an update that adds back in almost all currencies, sourced from here:http://www.floatrates.com/daily/usd.json Let me know if people have issues!

JKAbrams commented 4 years ago

The Bitcoin price is incorrect, Equate currently reports the price at $8740 while Bitstamp has $5519.

BTC is lacking from the .json you linked.

greatanotherusername commented 4 years ago

Second this ^. 1 BTC = $3,197.67 USD in the app; it should be twice this. The BTC/ETH rates also need updating (₿0.02, not ₿0.08).

EvanRespaut commented 4 years ago

Yeah the currency isn't updating anymore, so it's using the last value it has. I'm guessing the free API I was using is gone or not so free anymore. I'll try to find a replacement .

ghost commented 4 years ago

Someone please switch this to the Coin Paprika API. It's free and reliable, I've been using it for many months: https://api.coinpaprika.com/v1/tickers

And while you're at it, please make the top 200 coins / tokens available. If you want to make all tickers available then keep in mind some coins are using the same abbreviation. For this, simply choose the highest ranking.

Thanks

EvanRespaut commented 4 years ago

That API looks solid, I'll swap it over to that.

EvanRespaut commented 4 years ago

App is updated on play store with that new crypto source. Give it a day or so for Google to push it out to everyone.

ghost commented 4 years ago

And f-droid? Thanks.

EvanRespaut commented 4 years ago

I tagged the project with a new version, so F-droid should update too.

ghost commented 4 years ago

Thanks. Post a dash address to receive a small tip.

EvanRespaut commented 4 years ago

Thank you!

I actually found another small bug with the traditional currency updater, floatrates seems to reject HTTP and only accepts HTTPS so I released a new version that fixes that. The latest version should be 2.0. This should be live on google play and f-droid shortly.