yfinance icon indicating copy to clipboard operation
yfinance copied to clipboard

Added property getting company executives

Open nikolamilosevic86 opened this issue 2 years ago • 3 comments

I have added a functionality that reads company executives with their salaries and dates of births if available. The retrieved information comes in dataframe in property executives.

nikolamilosevic86 avatar Apr 25 '22 09:04 nikolamilosevic86

Happy to help with the review if appropriate.

ProgrammingDao avatar Jul 12 '22 08:07 ProgrammingDao

Hi Nikola. I'm willing to merge but have one request - move the fetching into it's own function e.g. Ticker._get_executives(). Each scrape url should have its own function.

ValueRaider avatar Nov 02 '22 15:11 ValueRaider

@ValueRaider if merged please make sure it does not affect performance of other API:s New features should be carefully added and be of common use as all new features need to be maintained and supported.

In my opinion before adding more features to the already bloated and buggy base.py the code need to restructured to allow for a more plugin oriented API.

fredrik-corneliusson avatar Nov 06 '22 23:11 fredrik-corneliusson