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 webhook endpoint, and other fixes #1766

Draft
wants to merge 19 commits into
base: dev
Choose a base branch
from
Draft

Add webhook endpoint, and other fixes #1766

wants to merge 19 commits into from

Conversation

ogabrielluiz
Copy link
Contributor

First simple implementation of a webhook feature.

#1732

Copy link

vercel bot commented Apr 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langflow ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 8:28pm

…int to provide clearer information on its purpose, arguments, and return value. Remove outdated request JSON structure and example request sections. Add information on the arguments and exceptions raised for better clarity.
@yamonkjd
Copy link
Contributor

I thought the webhook component would be processed later, but you seem like a genius. The code looks very Pythonic and well-written. Additionally, it would be great to add an unmodifiable "Copy" attribute with drag capabilities to the field_type in the frontend code, so users can easily copy the URL directly from the component. Of course, I think this type would be useful not only in this component but in all components where it can be applied.

@ogabrielluiz
Copy link
Contributor Author

ogabrielluiz commented May 23, 2024

I thought the webhook component would be processed later, but you seem like a genius. The code looks very Pythonic and well-written. Additionally, it would be great to add an unmodifiable "Copy" attribute with drag capabilities to the field_type in the frontend code, so users can easily copy the URL directly from the component. Of course, I think this type would be useful not only in this component but in all components where it can be applied.

hey @yamonkjd

How are you?

You are too kind. hahah

I'm not sure I understand the "Copy" attribute part. Could you elaborate?

I understand now and this is something I've been thinking about a lot lately. I'll think of a way to add that.

…unction name to createTabsArray for clarity

feat(constants.ts): add support for including webhook code in tabs array based on flow webhook property
feat(index.tsx): update function calls to use new createTabsArray function and include webhook code in tabs array based on flow webhook property

refactor(utils.ts): remove trailing commas in function parameters to improve code consistency and readability
feat(utils.ts): add support for including webhook cURL code and tweaks code in tabs array creation for better code organization and presentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants