-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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(platform): Support manually setting up webhooks #8750
base: dev
Are you sure you want to change the base?
Conversation
…e hooks + other infra
…inier/open-1961-implement-github-on-pull-request-block
…h/Node | Graph/Node -> GraphModel/NodeModel
…inier/open-1961-implement-github-on-pull-request-block
8795e62
to
088b131
Compare
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
transcriptions: list[Transcription] | ||
|
||
|
||
class CompassAITriggerBlock(Block): |
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.
Can this have a more specific/descriptive name?
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.
CompassAITranscriptionTriggerBlock?
def __init__(self): | ||
super().__init__( | ||
id="9464a020-ed1d-49e1-990f-7f2ac924a2b7", | ||
description="This block forwards an input value as output, allowing reuse without change.", |
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.
very generic
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.
fair
9cc9579
to
c54c52a
Compare
The active status icons don't appear here which is weird |
This is in the export
|
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
The webhooks system as is works really well for full blown enterprise webhooks managed via a UI. It does not work for more "chill guy" webhook tools that just send notifications sometimes.
Changes 🏗️
feat(blocks): Add Compass transcription trigger block
feat(backend): Amend webhooks system to support manual-set-up webhooks
ManualWebhookManagerBase
feat(frontend): Add UI to pass webhook URL to user on manual-set-up webhook blocks
refactor(backend): Rename
backend.integrations.webhooks.base
to._base
TODO
Checklist 📋
For code changes: