alleyway / add-tradingview-alerts-tool

Automated entry of TradingView alerts for bot trading tools such as 3Commas, Alertatron, CryptoHopper, etc.
MIT License
626 stars 140 forks source link

Unable to upgrade version #54

Closed ramnite closed 3 years ago

ramnite commented 3 years ago

I have been using ATAT for some time.

Yesterday ATAT stopped working so I decided to upgrade.

After installing node 14.16 and running npx @alleyway/create-tradingview-alerts-home I get the following error: npm ERR! Cannot read property 'spec' of undefined

Screen Shot 2021-05-10 at 10 33 43

mlake commented 3 years ago

Hi - thanks for reaching out...

having difficulty replicating this error.

is it possible you have an older version of npm installed?

npm --version

also, which OS are you using?

ramnite commented 3 years ago

Thanks for the fast response.

sh-3.2# npm --version
7.12.0

I am using macOS BigSur 11.3

ramnite commented 3 years ago

I have updated node using:

sudo npm cache clean -f
sudo npm install -g n
sudo n stable

and now, when I run "npx @alleyway/create-tradingview-alerts-home", I can see additional prompts. Give me a minute, I will see if the issue is solved and report back.

ramnite commented 3 years ago

It seems it was a problem with the node version.

After running the commands above, I have been able to create the "tradingview-alerts-home" directory.

I am able to fetch pairs. However, when I run add-alerts, I get ERROR Invalid csv file format.

As recommended on https://github.com/alleyway/add-tradingview-alerts-tool/issues/53, I try to run npm install @alleyway/add-tradingview-alerts-tool@1.8.2 but I get the following errors.

Screen Shot 2021-05-10 at 11 25 45

mlake commented 3 years ago

progress...

could you please post the .csv file you're using?

ramnite commented 3 years ago

Here they are.

pairs_excluded.csv pairs_included.csv

mlake commented 3 years ago

ah, okay - your .csv file for the pairs_included needs to have the following header. Will update the error message to be more clear...

symbol,base,quote
ramnite commented 3 years ago

That was it.

Now I am getting errors during the alert creation process of the first alert, but I think that's related to my config.yml. Maybe the config.yml format has changed in the last version/s? That's a different topic/issue. I will work on it now to see if I can fix it on my own.

Thank you very much. I am closing this ticket now.