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

Add Lark to notifications. #1137

Merged
merged 10 commits into from May 12, 2024
Merged

Add Lark to notifications. #1137

merged 10 commits into from May 12, 2024

Conversation

iuime
Copy link

@iuime iuime commented Dec 13, 2023

Add Lark to notifications.

Add Send Notifications to Lark
Add Send Notifications to Lark
Add Send Notifications to Lark
Add Send Notifications to Lark
Add Send Notifications to Lark
Add Send Notifications to Lark
Add Send Notifications to Lark
Update message format to support markdown
@AnonymousWP
Copy link
Collaborator

Can you please write a more detailed PR description? What's lark? What's the benefit?

@iuime
Copy link
Author

iuime commented Dec 13, 2023

Can you please write a more detailed PR description? What's lark? What's the benefit?

Lark is developed by ByteDance(TikTok's parent company) and is an application similar to an instant messaging tool(Slack). It is currently widely used in China and Asian countries. Since I've noticed that others share similar needs and would like to use Lark for notifications, I suggest merging this branch. Thank you.

Lark site:https://www.larksuite.com/en_us

@AnonymousWP
Copy link
Collaborator

Can you please write a more detailed PR description? What's lark? What's the benefit?

Lark is developed by ByteDance(TikTok's parent company) and is an application similar to an instant messaging tool(Slack). It is currently widely used in China and Asian countries. Since I've noticed that others share similar needs and would like to use Lark for notifications, I suggest merging this branch. Thank you.

Lark site:https://www.larksuite.com/en_us

So this PR brings support for sending notifications/messages to Lark?

@psyray
Copy link
Collaborator

psyray commented Dec 16, 2023

@iuime Everything has been tested and working on latest master branch ?

@iuime
Copy link
Author

iuime commented Dec 17, 2023

Can you please write a more detailed PR description? What's lark? What's the benefit?

Lark is developed by ByteDance(TikTok's parent company) and is an application similar to an instant messaging tool(Slack). It is currently widely used in China and Asian countries. Since I've noticed that others share similar needs and would like to use Lark for notifications, I suggest merging this branch. Thank you.
Lark site:https://www.larksuite.com/en_us

So this PR brings support for sending notifications/messages to Lark?

Yes, sending notifications/messages to Lark.

@iuime
Copy link
Author

iuime commented Dec 17, 2023

@iuime Everything has been tested and working on latest master branch ?

Yes, everything has been tested and is working on the latest master branch.

@AnonymousWP AnonymousWP added the Waiting for Merge Already Worked, waiting to merge label Feb 21, 2024
@yogeshojha yogeshojha changed the base branch from master to release/2.1.0 May 12, 2024 13:33
@yogeshojha
Copy link
Owner

Thank you for your PR.

Very good addition.

@yogeshojha yogeshojha merged commit d7641f5 into yogeshojha:release/2.1.0 May 12, 2024
3 of 5 checks passed
@yogeshojha yogeshojha removed the Waiting for Merge Already Worked, waiting to merge label May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants