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.

Sign up for GitHub

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

Jump to bottom

feat: implement telegram bot plugin #1740

Merged
merged 10 commits into from
May 26, 2023
Merged

feat: implement telegram bot plugin #1740

merged 10 commits into from
May 26, 2023

Conversation

athurg
Copy link
Contributor

@athurg athurg commented May 25, 2023

This PR add Telegram Bot support inside the Memos. Core features include:

  • Support both text and photo message, include multiple photo message.
  • One Telegram Bot need for whole Memos Server, every user can bind there Telegram UserID in UserSettings of Memos .
  • Pull mode of Telegram Bot, which works on your office LAN or home NAS.

Because it's a large PRs, so some features will be added in next version, such as:

  • Webhook mode of Telegram Bot, which for people who cannot touch Telegram API.
  • Support message other than text and photos, such as File Message, Location Message, etc.

@athurg athurg requested a review from boojack as a code owner May 25, 2023 13:35
@github-actions
Copy link

github-actions bot commented May 25, 2023 •
edited
Loading

Uffizzi Preview deployment-26647 was deleted.

@athurg athurg self-assigned this May 25, 2023
boojack
boojack reviewed May 25, 2023
View reviewed changes
Copy link
Collaborator

@boojack boojack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall. Please fix the linter errors

api/user_setting.go Outdated Show resolved Hide resolved
plugin/telegram/api_edit_message.go Outdated Show resolved Hide resolved
boojack
boojack approved these changes May 26, 2023
View reviewed changes
Copy link
Collaborator

@boojack boojack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT, let's move on!

@boojack boojack changed the title Feature: Telegram bot feat: implement telegram bot plugin May 26, 2023
@athurg
Copy link
Contributor Author

athurg commented May 26, 2023

I have rebase the branch into latest main, and fix all the golang static check failure.

@boojack boojack merged commit 1282fe7 into usememos:main May 26, 2023
@athurg athurg mentioned this pull request May 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boojack boojack boojack approved these changes

Assignees

@athurg athurg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@athurg @boojack