-
Notifications
You must be signed in to change notification settings - Fork 201
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
feat(service): add lark service #300
Conversation
const ( | ||
openID receiverIDType = "open_id" | ||
userID receiverIDType = "user_id" | ||
unionID receiverIDType = "union_id" | ||
email receiverIDType = "email" | ||
chatID receiverIDType = "chat_id" | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the receiverIDType
to all constants due to this. Could you please verify that this does not break functionality and/or behavior?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested it again manually and still works fine!
@immanuelhume so sorry for the very late response. I fixed the compatibility issues with the main branch and some linter warnings. Please check those before you push next time, please. I greatly appreciate your patience and contribution! |
@nikoksr Sorry I must've missed those warnings. Thanks for maintaining this project! |
Description
Adds a notifier service for Lark.
Motivation and Context
This addresses #268.
Lark has three kinds of bots, referred to as "webhook", "custom app", and "store app" bots. The first two are supported through this PR, which explains the two different files
webhook.go
andcustom_app.go
. The store apps have a tricky API which would work against the simple style of this library so they are not included here.How Has This Been Tested?
Basic unit tests using mocks have been written. Manual testing has also been performed.
Screenshots / Output (if appropriate):
The first screenshot is with a webhook. The second is with a custom app.
Types of changes
Checklist: