Skip to content

Updated yahoo.finance.keystatistics.xml#438

Open
JayDeeNZ wants to merge 1 commit into
yql:masterfrom
JayDeeNZ:patch-2
Open

Updated yahoo.finance.keystatistics.xml#438
JayDeeNZ wants to merge 1 commit into
yql:masterfrom
JayDeeNZ:patch-2

Conversation

@JayDeeNZ

Copy link
Copy Markdown

Work with changed xpath on Yahoo finance.

Work with changed xpath on Yahoo finance.
@BobAleena

Copy link
Copy Markdown

What is preventing this from being merged?

@JayDeeNZ

Copy link
Copy Markdown
Author

I believe either a moderator or the owner of the original code needs to approve the pull request.

In the meantime you can still use this fix in the YQL console like so:
use "https://raw.githubusercontent.com/yql/yql-tables/4404b415f2c5a2353966f7d054c238e29ef1a292/yahoo/finance/yahoo.finance.keystats.xml" as keystatistics; select * from keystatistics where symbol='MSFT';

@BobAleena

Copy link
Copy Markdown

@BobAleena

Copy link
Copy Markdown

Exactly what I needed -- i'm unblocked on the keystats table error now. Thanks!

@BobAleena

Copy link
Copy Markdown

Looks like something has changed to break this again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants