-
-
Notifications
You must be signed in to change notification settings - Fork 46.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated the URL and HTML tags for scrapping yahoo finance #8942
updated the URL and HTML tags for scrapping yahoo finance #8942
Conversation
for more information, see https://pre-commit.ci
@cclauss This is my first contribution will really appreciate some feedback |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for the most part, but I'll still have to verify whether the new URL works correctly myself when I get the chance.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verified that the fix works, LGTM
…hms#8942) * updated the url and tags for yahoo finance * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * updated to return the error text --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
The existing Yahoo URL for scrapping the stock price is no longer maintained from August 26th, 2021 (attached the screenshot below). Moreover there is a change in HTML tags for the current price field. I have added the new URL and have updated the HTML tags required to fetch the current price of the stock.
Checklist: