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

feat(platform): Support manually setting up webhooks #8750

Open
wants to merge 104 commits into
base: dev
Choose a base branch
from

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Nov 24, 2024

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

    • Make event filter input optional on webhook-triggered nodes
    • Make credentials optional on webhook-triggered nodes
    • Add code path to re-use existing manual webhook on graph update
    • Add ManualWebhookManagerBase
  • feat(frontend): Add UI to pass webhook URL to user on manual-set-up webhook blocks

    image

  • refactor(backend): Rename backend.integrations.webhooks.base to ._base

TODO

  • Pass webhook URL for manual webhooks to user in front end

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • ...

Pwuts and others added 30 commits October 17, 2024 00:11
…inier/open-1961-implement-github-on-pull-request-block
…inier/open-1961-implement-github-on-pull-request-block
@Pwuts Pwuts force-pushed the exploration/other-types-of-hooks branch from 8795e62 to 088b131 Compare December 9, 2024 21:37
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Dec 11, 2024
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Dec 11, 2024
@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end size/xl and removed size/l labels Dec 11, 2024
transcriptions: list[Transcription]


class CompassAITriggerBlock(Block):
Copy link
Member

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?

Copy link
Member Author

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.",
Copy link
Member

Choose a reason for hiding this comment

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

very generic

Copy link
Member Author

Choose a reason for hiding this comment

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

fair

@Pwuts Pwuts marked this pull request as ready for review December 12, 2024 14:17
@Pwuts Pwuts requested a review from a team as a code owner December 12, 2024 14:17
@Pwuts Pwuts requested review from Swiftyos and Bentlybro and removed request for a team December 12, 2024 14:17
@Pwuts Pwuts force-pushed the exploration/other-types-of-hooks branch from 9cc9579 to c54c52a Compare December 12, 2024 14:20
@ntindle
Copy link
Member Author

ntindle commented Dec 12, 2024

Works but overflows box for me
image

@ntindle
Copy link
Member Author

ntindle commented Dec 12, 2024

The active status icons don't appear here which is weird

@ntindle
Copy link
Member Author

ntindle commented Dec 12, 2024

This is in the export

 "webhook": {
        "id": "a6ba1a21-e2d2-40e3-8895-8e407fbe5b9d",
        "user_id": "eb3beaac-09ee-4f97-beee-1ac37f7dd93a",
        "provider": "compass",
        "credentials_id": "",
        "webhook_type": "transcription",
        "resource": "",
        "events": [],
        "config": {},
        "secret": "4d36c1efda70ac47d73de1e6462ab33d0778504a4d3e0cad8d63258c15919232",
        "provider_webhook_id": "",
        "attached_nodes": null,
        "url": "https://prompt-neat-flea.ngrok-free.app/api/integrations/compass/webhooks/a6ba1a21-e2d2-40e3-8895-8e407fbe5b9d/ingress"
      }
      ```

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Dec 13, 2024
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts platform/backend AutoGPT Platform - Back end platform/blocks platform/frontend AutoGPT Platform - Front end size/xl
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants