-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Feat/jotform #2616
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
base: staging
Are you sure you want to change the base?
Feat/jotform #2616
Conversation
… fixes, subflow resize clamping
…ribe, auth checks, new db indexes
…dioai#2481) The realtime service network policy was missing the custom egress rules section that allows configuration of additional egress rules via values.yaml. This caused the realtime pods to be unable to connect to external databases (e.g., PostgreSQL on port 5432) when using external database configurations. The app network policy already had this section, but the realtime network policy was missing it, creating an inconsistency and preventing the realtime service from accessing external databases configured via networkPolicy.egress values. This fix adds the same custom egress rules template section to the realtime network policy, matching the app network policy behavior and allowing users to configure database connectivity via values.yaml.
…rovements, additional kb tag types
…ts, LLM providers standardization
|
@Shubhamxshah is attempting to deploy a commit to the Sim Team on Vercel. A member of the Team first needs to authorize it. |
Greptile SummaryAdded comprehensive Jotform integration supporting both webhook triggers and API-based operations. Key Changes:
Technical Implementation:
Testing Confirmed: Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User as User (Jotform)
participant Jotform as Jotform Service
participant Webhook as Webhook Endpoint
participant Processor as Webhook Processor
participant Parser as multipart/form-data Parser
participant DB as Database
participant Queue as Task Queue
participant Executor as Workflow Executor
User->>Jotform: Submit form
Jotform->>Webhook: POST /api/webhooks/{path}<br/>(multipart/form-data)
Webhook->>Processor: parseWebhookBody()
Processor->>Parser: Parse Content-Type: multipart/form-data
alt Has rawRequest field
Parser->>Parser: JSON.parse(rawRequest)
else Direct form fields
Parser->>Parser: Convert FormData to object
end
Parser-->>Processor: Return parsed body
Processor->>DB: Find webhook by path
DB-->>Processor: Return webhook + workflow config
Processor->>Processor: checkWebhookPreprocessing()<br/>(rate limits, deployment)
Processor->>Queue: Queue webhook execution
Queue-->>Webhook: Return 200 OK
Queue->>Executor: Execute workflow with webhook.data.payload
Executor->>Executor: Access form fields<br/>(e.g., webhook.data.payload.q3_email1)
|
Greptile found no issues!From now on, if a review finishes and we haven't found any issues, we will not post anything, but you can confirm that we reviewed your changes in the status check section. This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR". |
|
@waleedlatif1 @icecrasher321 @emir-karabeg please review this. |
Added https://jotform.com as a tool and trigger/block integration. This enables workflows to:
Key implementation details:
Type of Change
Testing
How has this been tested?
What should reviewers focus on?
Checklist
Screenshots/Videos

1.