Conversation
|
Review this PR on mrge.io |
|
@ivanma9 is attempting to deploy a commit to the Sim Studio Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
.../w/[id]/components/workflow-block/components/sub-block/components/webhook/webhook-config.tsx
Outdated
Show resolved
Hide resolved
| "uuid": "^11.1.0", | ||
| "zod": "^3.24.2" | ||
| "zod": "^3.24.2", | ||
| "zustand": "^5.0.3" |
There was a problem hiding this comment.
Zustand is already used extensively in the codebase but was missing from package.json dependencies, suggesting this PR is fixing an undeclared dependency.
...orkflow-block/components/sub-block/components/webhook/components/providers/twilio-config.tsx
Outdated
Show resolved
Hide resolved
...orkflow-block/components/sub-block/components/webhook/components/providers/twilio-config.tsx
Outdated
Show resolved
Hide resolved
...mponents/workflow-block/components/sub-block/components/webhook/components/webhook-modal.tsx
Outdated
Show resolved
Hide resolved
|
@ivanma9 please resolve the merge conflicts and make the PR again |
2f7f9ba to
7604ecb
Compare
68794b3 to
41c5fb7
Compare
this is my file for /simstudio/sim/app/api/webhooks/trigger/[path]/route.ts i noticed when i went to resolve my conflicts for the second time the twilio integration was gone, so here it is for reference because I am not sure if you meant to remove it. specifically the are with "else if (foundWebhook.provider === 'twilio') {" |
df6ba40 to
02657ec
Compare
9a2bfbc to
4540c41
Compare
a13b454 to
cc6299c
Compare
628a9c3 to
d8025a2
Compare
a18a617 to
b29827c
Compare
1b3f825 to
b9b662b
Compare
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Solves issue #212. I wanted to make a way to trigger worflows that start from an SMS.
Fixes #212 (issue)
Type of change
With the correct configuration for Twilio, can trigger webflow from webhook via SMS
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
Tested through logs and listeners from a registered Twilio account to test the phone number.
Make an account through Twilio. Make sure to set the ngrok url with the webhook endpoint from the twilio configuration on simstudio.
Send an SMS to that number.
Checklist:
npm test)Security Considerations:
Additional Information:
Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.