lavolp3 / MMM-AVStock

MagicMirror module for displaying stock price with yfinance API
MIT License
38 stars 18 forks source link

API Broken? #62

Closed Dresch360 closed 1 year ago

Dresch360 commented 1 year ago

Looks like the stocks are no longer displaying. Wondering if AlphaVantage revised their API?

hermannw commented 1 year ago

I noticed that too...

bclinton1 commented 1 year ago

Same here....

gnurph69 commented 1 year ago

AlphaVantage now saying this is a premium API call.

bclinton1 commented 1 year ago

Thanks for the info. Time to look for something else I suppose :)

SleipnirMS commented 1 year ago

When using chartInterval = daily, you have to set it to "daily_adjusted" and replace both "daily" entries in node_helper.js to "daily_adjusted".

Then it uses the free api part and everything is fine again.

hermannw commented 1 year ago

It works indeed! Thank you very much!

bclinton1 commented 1 year ago

When using chartInterval = daily, you have to set it to "daily_adjusted" and replace both "daily" entries in node_helper.js to "daily_adjusted".

Then it uses the free api part and everything is fine again.

Awesome. Thanks for sharing this.

Dresch360 commented 1 year ago

Thank you so much! This worked.

lavolp3 commented 1 year ago

I don't really know why but it works for me with the "daily" function, but not with the "daily_adjusted" function. So try to change it back or do a git reset.

The error only seemed to appear for me when calling technical parameters (in my case SMA20) with the call. So, daily_adjusted with technical parameters seems to be a premium function. "Daily" calls seem to work