-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Patch keyError in #2027 #2029
base: dev
Are you sure you want to change the base?
Patch keyError in #2027 #2029
Conversation
…rsion Fix datetime conversion with mixed timezones when ignore_tz is False
That's not the solution. Look at the metadata. |
The same information as in the original PR, #2026, can be fetched from the
will have the same format as most other stocks for this key. I don't know if Yahoo Finance may give differently formated metadata for the ´'currentTradingPeriod'´ key. |
Just add 2 unit tests: one with prepost=False, other with True. Then rebase to dev branch #1084 |
Solves ´KeyError´ occurrence in #2027.
Adds Try/Except statement to find keyerror and return to old code if the keyerror is triggered.