plutov / formulosity

Self-hosted Surveys as Code platform.
https://formulosity.vercel.app
MIT License
181 stars 20 forks source link

first draft with webhook support #25

Closed krishnateja262 closed 3 weeks ago

krishnateja262 commented 1 month ago

@plutov raising a draft PR, let me know if this is ok or any changes that are required. Working on UI changes and timeout setting.

vercel[bot] commented 1 month ago

@krishnateja262 is attempting to deploy a commit to the formulosity Team on Vercel.

A member of the Team first needs to authorize it.

plutov commented 1 month ago

Hi @krishnateja262 yes the format is good. The yam configuration is enough for now, so we just need to make it work:

plutov commented 1 month ago

Hi @krishnateja262 how are things going with this PR? do you need help maybe?

krishnateja262 commented 1 month ago

Hi @plutov, sorry got occupied with work, pushed few more changes, for this PR i created a table for storing webhook responses and status codes when running the application in docker the new table is not getting created getting this error no such table: surveys_webhook_responses

plutov commented 1 month ago

Hi @krishnateja262 I see that you've added the CREATE TABLE into the existing migration, it's better to add to a new one. Because once the migration is executed once it won't be executed anymore, that's how migrations work.

krishnateja262 commented 1 month ago

Hi @plutov i have made the changes, attaching screenshot for the UI change, please review and let me know for any changes. The downloaded file also has the webhook data

Screenshot 2024-10-13 at 10 14 19 PM
plutov commented 3 weeks ago

This is awesome @krishnateja262 will test it in the next few days and probably merge it!

plutov commented 3 weeks ago

it works well, I am merging it!