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

Implemented feature to get quarterly and annual financial data of stock #29

Closed
wants to merge 2 commits into from

Conversation

dhruvin-084
Copy link

Function getFinancialData returns list of dictionaries, which contains various financial data such as Revenue, OtherIncome, TotalIncome, Expenditure, Interest, PBDT, Depreciation, PBT, Tax, NetProfit, Equity, EPS, CEPS, OPM, NPM from last five quarter or last five years.

Please review it and let me know if any thing else can be added or improved.

@codecov
Copy link

codecov bot commented Sep 14, 2021

Codecov Report

Merging #29 (47f9266) into master (4f5f313) will decrease coverage by 7.42%.
The diff coverage is 15.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   65.26%   57.84%   -7.43%     
==========================================
  Files           8        9       +1     
  Lines         262      306      +44     
==========================================
+ Hits          171      177       +6     
- Misses         91      129      +38     
Impacted Files Coverage Δ
bsedata/financialData.py 11.90% <11.90%> (ø)
bsedata/bse.py 90.69% <66.66%> (-1.99%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f5f313...47f9266. Read the comment docs.

@sdabhi23
Copy link
Owner

BSE India has increased the security on it's APIs and this way of fetching data no longer works

@sdabhi23 sdabhi23 closed this Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants