yql / yql-tables

YQL is extensible using a table definition. This repository will hold community contributed definitions.
http://developer.yahoo.com/yql
724 stars 416 forks source link

yahoo.finance.balancesheet URL needs fixed #346

Open mediocrepat opened 11 years ago

mediocrepat commented 11 years ago

I had originally posted this under the yahoo.finance.balancesheet fix, but it appears that issue is closed and I wanted to ensure this issue is open. Apologies for the crosspost.

It appears yahoo.finance.balancesheet is broken. It's trying to pull on the below URL, which currently returns a 404.

http://finance.yahoo.com/q/?s=aapl&quarterly

Correct URL should be (example given for AAPL): Quarterly: http://finance.yahoo.com/q/bs?s=AAPL

Annually: http://finance.yahoo.com/q/bs?s=AAPL&annual

I suspect CashFlow and IncomeStatement are also broken now too. If this gets fixed, I'll post on those respective files.

Would love a fix for this!

tmedford commented 11 years ago

I would love to see this fix as well.

UserABC commented 11 years ago

I would also love to see this fixed as well, although it seems very unlikely - looking at the history of this. It is frustrating that Yahoo Finance has changed the URLs, so that the code breaks :-( I fear they have made other changes, which makes the fixing task more than just a matter of updating the URL build up in the code :-(

ryanh-ai commented 11 years ago

Check out gist below until the pull request gets approved:

https://gist.github.com/canada4663/7518842

Should work - let me know if not and I'll fix it.

UserABC commented 11 years ago

Great - thanks! Looking forward to the approval of the pull request.

UserABC commented 10 years ago

Thx canada4663 - the pull request finally went through and the YQL console for balancesheet now works nicely :-)

ryanh-ai commented 10 years ago

no problem!!

do you know any way of scraping / extracting the currency and/or exchange from a symbol on yahoo.finance?

Ryan

On Dec 14, 2013, at 11:33 AM, UserABC notifications@github.com wrote:

Thx canada4663 - the pull request finally when through and the YQL console for balancesheet now works nicely :-)

— Reply to this email directly or view it on GitHub.

Abeabe commented 10 years ago

This is currently broken (2014-08-21)

It returns an error: "Exception while converting tidy sanitized html to Dom"

I'm not sure if the code needs to be fixed or if Yahoo need to fix YQL itself?

Anyone?

Could this https://github.com/trunghung/yql-tables/commit/99094edfee763169075314e35cfcda62fc9c7ecf be causing the issue??

PS: Also applies to finance.incomestatement and finance.cashflow.