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

Search URL broken #45

Open
mash420 opened this issue Apr 30, 2024 · 3 comments
Open

Search URL broken #45

mash420 opened this issue Apr 30, 2024 · 3 comments

Comments

@mash420
Copy link

mash420 commented Apr 30, 2024

the search URL seems to be broken. My code no longer works when using it. get_hist works only if you supply the correct exchange:symbol now.

dietmarb01 added a commit to dietmarb01/tvdatafeed that referenced this issue May 5, 2024
 - Tradingview have changed their search API to "v3". In addition,
   their Nginx frontend now blocks requests with Origin+Referer other
   than their own site.
 - fixes rongardF#45
 - version bumped to 2.1.0_issue45
 - only tested with Python 3.12

Changes in this commit:
	modified:   __init__.py
	modified:   main.py
@dietmarb01
Copy link

Tradingview have changed their search API to "v3".
The query now wraps the resulting 'symbols' list in a dict and also returns an integer 'symbols_remaining' (maybe for paging?).
In addition, their Nginx frontend now blocks requests with Origin+Referer other than their own site.

A fix is available at dietmarb01/tvdatafeed
NOTES:

  • Tested with Python 3.12 only
  • Basic functionality is restored; the additional return value is not processed
  • Version upped to 2.1.0_issue45 -- the source repo is at 2.1.0, while the PyPI package tradingview-datafeed is at 2.1.1?!

A formal pull request is in the making, but I won't be able to provide it for the next few weeks due to absence. Volunteers welcome to take over...

@dietmarb01
Copy link

@demonarch: Please Update the source attribute on PyPI to the repo that corresponds to the published version tradingview-datafeed-2.1.1. Currently it refers to this repo which is at 2.1.0; also your own (public) fork is at 2.1.0.

Thanks, Dietmar

@demonarch
Copy link

dear @TradingBias ,
Can you please explain your issue with more details?

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

No branches or pull requests

3 participants