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

Fixed /bse/getIndexInfo?indexId=16 issue #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nikkhil31
Copy link

Due to 301 response This API( http://localhost:3000/bse/getIndexInfo?indexId=16 ) was not working.
I have fixed this issue using BSE's another API.

now it's working fine.

@nikkhil31
Copy link
Author

I'm also fixing other API's meanwhile I'm willing to review this pull request from the maintener.

@maanavshah
Copy link
Owner

@nikkhil31 Kudos to your awesome work! Please let me know once you are done so I can merge the PR.

@maanavshah
Copy link
Owner

@nikkhil31 Are you done with all the changes?

@pramodshetty001
Copy link

pramodshetty001 commented Aug 22, 2022

BSE broken again.

NSE works fine

Might be a minor scraping fix, but I dont understand node.js.

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