search

thibaultyou / tradingview-alerts-processor

Minimalist service designed to execute TradingView webhooks and process them to cryptocurrencies exchanges.
MIT License
79 stars 23 forks source link

:package: Update package to reenable binanceus #34

Closed leifjones closed 3 years ago

leifjones commented 3 years ago

This change is working for me (vs current image) when I built and pushed to lsegen/tradingview-alerts-processor

thibaultyou commented 3 years ago

Thank for this update @mrsegen 🙏.

I'll merge this, rebuild and push this update to Docker.io in a few hours.

leifjones commented 3 years ago

I had debugged locally. At first I wondered if it was an issue with JsonDb, but it seemed the root error was somewhere deep in ccxt. Given how rapidly ccxt releases new versions, I never pinned down specfic fix was on their end. But that's how packages work! Glad we don't have to resolve it ourselves!

thibaultyou commented 3 years ago

I was not even aware that we had issues with Binance since I've not much time to see our Discord channel.

Thanks again for this update.

thibaultyou commented 3 years ago

I really need to add Docker image build in the CI so you don't need me to add new things ;)

thibaultyou commented 3 years ago

Docker image is now updated 🙏

thibaultyou commented 3 years ago

I really need to add Docker image build in the CI so you don't need me to add new things ;)

@mrsegen done here, you should be able to refresh Docker image at every merge / push on main branch.