taoteh1221 / Open_Crypto_Tracker

Bitcoin / Alts private portfolio tracker, with email / text / alexa / telegram price alerts, charts, leverage support and much more.
https://taoteh1221.github.io
GNU General Public License v3.0
92 stars 22 forks source link

Facing issue with few coin pairs #36

Closed s1-ranjan closed 2 years ago

s1-ranjan commented 2 years ago

I tried to add few coins in this app, almost most of the coins are working normally over here but few coins are showing some error as shown below: [2021-10-12 19:30:06] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "uni-7";
[2021-10-12 19:30:09] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "bnb-2";
[2021-10-12 19:30:11] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "ufii";
[2021-10-12 19:30:11] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "ufii-2";
[2021-10-12 19:30:15] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "aave";
[2021-10-12 19:30:15] cron => market: oct_asset->charts_price_alerts() - No INR conversion value (INR pairing) for "aave-2"; I am attaching my config file over here please have a look at the configuration and please suggest any corrections in the configuration settings. config.php.zip .

taoteh1221 commented 2 years ago

The was a weird duplicate entry in zebpay's API results for some coins like UNI, so I upgraded the API call in the app to look for this. So zebpay should be fixed now in the next release.

As for some of your other issues shown in the error log, the coin symbol does not exist on that exchange's API (UFII is not an API result at wazirix or bitbns), or it's under a different market name. I see no 'ufii' at these API endpoints for wazirix / bitbns:

https://bitbns.com/order/getTickerWithVolume

https://api.wazirx.com/api/v2/tickers

You probably should always double-check the market ID exists for any markets you add in the app, especially before adding charts / price alerts configurations. After you add each market, upload the new config, and test that market in the browser before adding charts / alerts for it.

taoteh1221 commented 2 years ago

I'm presuming this is resolved, closing for now.