Skip to content

feat: Add feishu servicedesk provider #5430

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
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: Add feishu servicedesk provider #5430

wants to merge 4 commits into from

Conversation

@Nctllnty
Copy link

@Nctllnty Nctllnty commented Nov 3, 2025 •
edited
Loading

This PR adds a new Provider to Keep AIOps: [feishu-servicedesk-provider], Closes: #5427

Related Issue: feat/feishu-servicedesk-provider

image image

@vercel
Copy link

vercel bot commented Nov 3, 2025

@Nctllnty is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Nov 3, 2025
@CLAassistant
Copy link

CLAassistant commented Nov 3, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 3, 2025

Skipped: This PR does not target one of your configured branches: (refactor/2512-nextjs-15)

@dosubot dosubot bot added Documentation Improvements or additions to documentation Feature A new feature Provider Providers related issues labels Nov 3, 2025
@Nctllnty
Copy link
Author

Nctllnty commented Nov 10, 2025

Can someone approve my PR application?

@talboren
Copy link
Member

talboren commented Nov 10, 2025

Can someone approve my PR application?

Can you please update Chinese to English?

@Nctllnty
Copy link
Author

Nctllnty commented Nov 10, 2025

Can someone approve my PR application?

Can you please update Chinese to English?
Of course, I will complete it.

@Nctllnty
Copy link
Author

Nctllnty commented Nov 11, 2025

@talboren The Chinese text in the documentation and code has been translated into English.

@shahargl
Copy link
Member

shahargl commented Nov 12, 2025

Hey! thanks for opening this PR!

We have some problem with the CI/CD. Going to handle it soon and let this PR in :)

@shahargl shahargl self-requested a review November 16, 2025 07:51
@shahargl
Copy link
Member

shahargl commented Nov 16, 2025

@Nctllnty tests failing (see UT's, docs, examples, etc) - do you want to solve it?

@Nctllnty
Copy link
Author

Nctllnty commented Nov 21, 2025

Hi @shahargl, I've fixed the linting errors and added the required docs, examples, and tests. The CI checks should be green now. Ready for re-review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shahargl shahargl Awaiting requested review from shahargl

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Documentation Improvements or additions to documentation Feature A new feature Provider Providers related issues size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[🔌 Provider]: lark/feishu servicedesk provider

4 participants

@Nctllnty @CLAassistant @talboren @shahargl