alvarobartt
commented
5 years ago T05 has been solved in ce99e95179a0c143f12d9fbfdbf3171a6da45a92, via adding the country param to retrieval functions as described in the issue and so on, the solution has been also described in the referenced commit.
So to extend the usage of the package, the retrieval of all the equities indexed in Investing.com is proposed, as currently investpy just supports spanish equities, which limits its usage since there are a lot of markets available.
📄 So on, some tasks can be defined in order to get to develop this enchancement and so to integrate it with the package:
[x] T01 - Research on which id fields are sent on the GET request to Investing.com
[x] T02 - Retrieve a listing with all the available countries which have equities listed in Investing and define a function to wrap it.
[x] T03 - Extend
investpy.equities.retrieve_equitiesfunctionallity and test its usage.[x] T04 - Check if there is any missing data in
equities.csvdue to connection errors since the scrapping process took too long and made a huge amout of consecutive requests to Investing.com. (Tip: define a function to wrap that check because it can happen again ifequities.csvis re-generated.)[x] T05 - Once
equities.csvis generated, add country param to equity retrieval functions since it is a new filtering needed to avoid duplicated equity names in different countries. (Note: this tasks is pretty similar to the one done for ETFs when country filtering was added)[x] T06 - Generate the docs and update investpy usage.
Additional tasks will be added to the previous listing if needed in order to complete this issue.