Skip to content
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

Allow data download from Poland #597

Merged
merged 2 commits into from Mar 13, 2019

Conversation

@MicBan
Copy link
Contributor

MicBan commented Nov 26, 2018

Current version of the code is very US focused while stooq is a Polish website. To download data from the WSE one need to provide a ticker only (such as PZU), website returns null if one add PL at the end. My small addition copes with this issue while still allowing US focus (i.e. use of ticker only approach to shares listed in US).

  • closes #xxxx
  • tests added / passed
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • added entry to docs/source/whatsnew/vLATEST.txt
MicBan and others added 2 commits Nov 26, 2018
Current version of the code is very US focused while stooq is a Polish website. To download data from the WSE one need to provide a ticker only (such as PZU), website returns null if one add PL at the end. My small addition copes with this issue while still allowing US focus (i.e. use of ticker only approach to shares listed in US).
@bashtage bashtage merged commit c714918 into pydata:master Mar 13, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.