-
Notifications
You must be signed in to change notification settings - Fork 1k
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: Slack Integration #2125
feat: Slack Integration #2125
Conversation
…nd-ui feat: Initialised OAuth integration with Slack for Slack-App
Slack oauth persistence
…pen id connection
Feat/slack formbricks bridge
@Palanikannan1437 is attempting to deploy a commit to the formbricks Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for following the naming conventions for pull request titles! 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gupta-piyush19 Thanks for making this PR ready 😊💪 I still found a few things in the code that needs a change and a few smaller issues with the functionality:
apps/web/app/(app)/environments/[environmentId]/integrations/slack/lib/slack.ts
Outdated
Show resolved
Hide resolved
...app/(app)/environments/[environmentId]/integrations/slack/components/AddIntegrationModal.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/(app)/environments/[environmentId]/integrations/page.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/(app)/environments/[environmentId]/integrations/slack/components/Connect.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/(app)/environments/[environmentId]/integrations/slack/page.tsx
Outdated
Show resolved
Hide resolved
…piyush/slack-app
…formbricks into piyush/slack-app
Thanks for the review, @mattinannt 🚀 |
…piyush/slack-app
…piyush/slack-app
@gupta-piyush19 for the kamal setup we also now need the environment variables in these files:
|
Thanks, @mattinannt, for the review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Palanikannan1437 @gupta-piyush19 Thanks for your great job! Works great and is ready to be merged! 🚀💪
Thanks, @Palanikannan1437 for the PR and @mattinannt for the review(s). 🙌 |
This doesn't show up for us in self-hosted. We are using https:// and have set up the variables. version v1.6.1. I follow documentation on website: https://formbricks.com/docs/integrations/slack#setup-in-self-hosted-formbricks |
@briannezhad Slack integration is yet to be released for self-hosted users in the upcoming release (in a few days). It is currently available for cloud users. We updated the documentation, which caused the confusion. Apologise for the inconvenience. You'll be notified once it's released. |
Thank you. 🙏 |
What does this PR do?
This PR adds in Slack integration to pipe in survery responses into your desired Slack Channels.
Demo
Screen.Recording.2024-01-22.at.5.37.53.PM.mov
How should this be tested?
https://localhost:3000/api/auth/callback/slack
as your redirect URLSLACK_CLIENT_ID
andSLACK_CLIENT_SECRET
in your .env fileAppreciated