Update Slack integration docs for marketplace compliance#1133
Update Slack integration docs for marketplace compliance#1133joelaguero merged 3 commits intodevelopfrom
Conversation
- Add privacy policy link as required by Slack reviewers - Add comprehensive AI disclaimer explaining AI components and usage - Add pricing information section with link to pricing page - Enhance permissions section with detailed scope explanations - Clarify mpim:read, chat:write.customize, and users:read.email usage - Address private channel name anonymization in privacy section Addresses Slack marketplace feedback items #2, #4, #5, #7, and #8
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. 🚀 New features to boost your workflow:
|
|
I see a check failed - I'm on it! 🫡 |
Remove extra asterisks in mpim:read permission description that were causing test failures.
|
✅ Fixed failing check in this commit. The issue was a markdown syntax error in the slack.mdx file - there were extra asterisks ( 💻 View my work • React 👍 or 👎 |
docs/integrations/slack.mdx
Outdated
| - **Send messages** - To communicate responses and provide updates | ||
|
|
||
| ### Enhanced Communication Features | ||
| - **Customize message appearance** (`chat:write.customize`) - Allows Codegen to post messages with custom usernames and avatars when representing different contexts or providing specialized responses (e.g., when acting as a specific type of assistant or when providing updates from different integrated services) |
There was a problem hiding this comment.
@codegen remove this bullet about Customize message appearance
There was a problem hiding this comment.
I'll remove that bullet about customizing message appearance! ✅
docs/integrations/slack.mdx
Outdated
| ### Why These Permissions Are Necessary | ||
|
|
||
| - **Email mapping** enables secure account linking between Slack and Codegen, ensuring proper access control | ||
| - **Custom appearance** allows for clearer communication when Codegen is providing updates from different integrated services |
There was a problem hiding this comment.
@codegen remove this bullet point mentioning custom appearance
docs/integrations/slack.mdx
Outdated
|
|
||
| - **Message Analysis:** Your Slack messages are processed by AI models to understand intent and generate appropriate responses | ||
| - **Code Context:** When working with repositories, AI models analyze relevant code to provide accurate assistance | ||
| - **Learning and Improvement:** AI interactions help improve response quality, but no personal data is used for model training without explicit consent |
There was a problem hiding this comment.
@codegen Remove the learning and improvement bullet here
There was a problem hiding this comment.
✅ Done! I've removed both requested items:
- Removed the "Customize message appearance" bullet point from the Enhanced Communication Features section
- Removed the "Custom appearance" reference from the permissions rationale
- Removed the "Learning and Improvement" bullet point from the AI Data Processing section
The changes have been pushed to PR #1133. The documentation now focuses on the core permissions and AI functionality without the customize appearance and learning improvement aspects.
💻 View my work • React 👍 or 👎
- Removed chat:write.customize scope explanation per feedback - Removed custom appearance reference from permissions rationale - Removed AI learning/improvement explanation per feedback
|
🎉 This PR is included in version 0.56.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This PR updates the Slack integration documentation to address feedback from Slack marketplace reviewers and ensure compliance with their requirements.
Changes Made
✅ Privacy Policy Link (Feedback #4)
✅ AI Disclaimer (Feedback #5)
✅ Pricing Information (Feedback #8)
✅ Enhanced Permissions Documentation (Feedback #7)
mpim:read- For group DM functionalitychat:write.customize- For custom usernames/avatars when representing different contextsusers:read.email- For mapping Slack accounts to Codegen accounts for proper authentication✅ Privacy Enhancements (Feedback #2)
Slack Marketplace Feedback Addressed
This PR directly addresses the following feedback items from Slack reviewers:
chat:write.customizeandusers:read.emailRemaining Technical Issues
The following items require code changes (not documentation) and are outside the scope of this PR:
mpim:readscope in OAuth URL (technical implementation)mpim:readscope (technical implementation)Files Changed
docs/integrations/slack.mdx- Updated with all compliance requirements💻 View my work • About Codegen