Fixed telegram query url

CyberPunkMetalHead / gateio-crypto-trading-bot-binance-announcements-new-coins

This is a crypto trading bot that scans the Binance Annoucements page for new coins, and places trades on Gateio

MIT License

1.21k stars 302 forks source link

Fixed telegram query url #164

Closed Linus045 closed 2 years ago

Linus045 commented 2 years ago

replaced multi-line string with concatenated single-line strings

As noticed by @warren-ru here: https://github.com/CyberPunkMetalHead/gateio-crypto-trading-bot-binance-announcements-new-coins/commit/9de541baab4cf5622201306954bc4d1dc2c02a5a#r65068689

The telegram query url was invalid.

Linus045 commented 2 years ago

[x] Testing required

Can someone with telegram set up test this branch and validate that everything works fine now? Otherwise I will try to get to it some time next week, quite busy at the moment.

warren-ru commented 2 years ago

Just tested. Works fine. As a piece of advice. They say it's better to include f in every line for consistency/readability.

Linus045 commented 2 years ago

Thanks for verifying. I will merge this then.

I don't think it really matters with the f-string since there is no parameter used inside the string. Also not sure if black (the formatter) wouldn't just remove it.