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

New feature request - add "expense ratio" #801

Closed
moedi85 opened this issue Jul 30, 2021 · 3 comments
Closed

New feature request - add "expense ratio" #801

moedi85 opened this issue Jul 30, 2021 · 3 comments

Comments

@moedi85
Copy link

moedi85 commented Jul 30, 2021

It would be great if "expense ratio" from the Yahoo Finance Summary tab would be added to the info that can be obtained from .info

@carlos-ventura
Copy link

carlos-ventura commented Apr 16, 2022

Since this feature has been long due and nothing is changing just fork the repo
Edit base.py file and add the following after self._info = {} is defined:

        expenseRatio = (data
                        .get('fundProfile', {})
                        .get('feesExpensesInvestment', {})
                        .get('annualReportExpenseRatio', None))

        try:
            self._info['annualReportExpenseRatio'] = expenseRatio
        except Exception:
            pass

@Courvoisier13
Copy link

This should be implemented. it's very easy as well.

@ValueRaider
Copy link
Collaborator

@moedi85 @carlos-ventura @Courvoisier13 Ran has granted me permissions to merge PRs etc, but help is appreciated in creating and/or reviewing PRs - anyone can do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants