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

Telegram bee updates #324

Closed
wants to merge 6 commits into from
Closed

Telegram bee updates #324

wants to merge 6 commits into from

Conversation

rubiojr
Copy link
Collaborator

@rubiojr rubiojr commented Jun 16, 2020

This a backwards compatible change, existing installs shouldn't note the difference.

Changes

Switch to tucnak/telebot

Nice, cleaner API and they seem to be doing a better job at releasing software.

Optional HTML parsing for messages

New option to enable HTML formatting, implementing a feature request in #308 (disabled by default). cc @it-parasite

rubiojr added 6 commits June 16, 2020 13:25
The API is nicer and they seem to be doing a better job at releasing
software (tagging versions, regular updates, etc).

This a backwards compatible change, existing installs shouldn't note
the difference.

I also run `go mod tidy` which resuled in a slightly bigger diff.
@muesli
Copy link
Owner

muesli commented Jun 17, 2020

Nice! Only downside I see right now is the new package requiring Go 1.12+. I'm happy to bump that though.

@rubiojr
Copy link
Collaborator Author

rubiojr commented Jun 18, 2020

@muesli 👍 up to you. We can keep this open till Go is bumped. Or I can do that too, let me know.

@muesli
Copy link
Owner

muesli commented Jul 4, 2020

Yeah, let's just bump it. I'll adjust the CI first, then we can re-run the tests and merge this.

@muesli
Copy link
Owner

muesli commented Oct 3, 2020

Sorry for the conflicts I introduced 🙈

@rubiojr
Copy link
Collaborator Author

rubiojr commented Oct 4, 2020

I'll have a look!

@rubiojr
Copy link
Collaborator Author

rubiojr commented Jun 14, 2022

This work is happening in Beehive's rewrite (#390 (comment)), so closing as this will (likely) never be addressed here.

@rubiojr rubiojr closed this Jun 14, 2022
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

Successfully merging this pull request may close these issues.

2 participants