tradingview / charting-library-tutorial

This tutorial explains step by step how to connect your data to the Charting Library
MIT License
405 stars 310 forks source link

Resolving symbol doesn't support resolving by short name #15

Open bhp-work opened 3 years ago

bhp-work commented 3 years ago

First of all thank you so much team for the ready to use tradingview implementation. This is really great.

It throws 'cannot resolve symbol' when compared. Also I am not sure if this issue is with all or just me because I did some code changes and not really sure if that has cause the issue. Please try to reproduce and check. If you are not able to reproduce then please close the issue :)

crp

timocov commented 3 years ago

I guess we have to set ticker property here as well. Would you like to make a PR?

timocov commented 3 years ago

And don't forget about https://github.com/tradingview/charting-library-tutorial/blob/master/documentation/datafeed-implementation.md#resolvesymbol

timocov commented 3 years ago

See https://github.com/tradingview/charting_library/wiki/Symbology#ticker.

Also, we can solve the issue by search not by just full_name, but by the name either.