Skip to content

Docs: Add comprehensive Social Media Integration Guide#370

Closed
firaseth wants to merge 3 commits intosipeed:mainfrom
firaseth:main
Closed

Docs: Add comprehensive Social Media Integration Guide#370
firaseth wants to merge 3 commits intosipeed:mainfrom
firaseth:main

Conversation

@firaseth
Copy link
Copy Markdown

📝 Integration Guide Update

This PR adds a comprehensive Social Media Integration Guide to the documentation, covering platform setups that were previously missing or only covered briefly.

🚀 Changes

  • New File: Created docs/social_media_guide.md with detailed, step-by-step configuration instructions for:
    • WhatsApp (Docker bridge setup)
    • Feishu / Lark (Custom app & event subscription)
    • Telegram, Discord, QQ, DingTalk, LINE, and Slack.
  • Updated README: Added a direct link to this guide under the "Chat Apps" section in the main README.md to help users find advanced configuration steps easily.

🎯 Value

This guide aims to reduce setup friction for users trying to connect PicoClaw to more complex platforms like WhatsApp and Feishu, ensuring they have a single source of truth for all channel configurations.

Copy link
Copy Markdown

@nikolasdehor nikolasdehor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR does not contain what the description claims. The PR description says it adds a comprehensive Social Media Integration Guide (docs/social_media_guide.md) and updates the README with a link, but the actual diff only adds a single file called PicoClaw-guide containing:

git am < path/to/picoclaw-docs.patch

This is a git command, not documentation. The described files (docs/social_media_guide.md, README.md update) are not present in the diff at all.

It looks like the actual documentation content was prepared as a git patch file but never applied. The PR should contain the actual markdown documentation, not a shell command referencing a local patch file.

Please recreate this PR with the actual documentation content committed.

@firaseth
Copy link
Copy Markdown
Author

firaseth commented Feb 20, 2026 via email

Added comprehensive guide for integrating PicoClaw with various social media platforms including Telegram, Discord, WhatsApp, WeChat, Feishu, QQ, DingTalk, LINE, and Slack.
Add PicoClaw Social Media Integration Guide
@sipeed-bot sipeed-bot bot added type: documentation Improvements or additions to documentation domain: channel labels Mar 3, 2026
Copy link
Copy Markdown

@nikolasdehor nikolasdehor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The content is thorough and well-organized, covering all the major messaging platforms. However, there are issues that need to be fixed:

  1. File naming and extension: The file is named 'PicoClaw-guide' with no extension. It should be 'PicoClaw-guide.md' since it contains Markdown content. Without the .md extension, GitHub will not render it properly.

  2. File location: A guide this large (780 lines) should go in a docs/ directory rather than the repository root. The root is already getting cluttered with multiple README translations.

  3. Potential security concern: The guide includes placeholder API keys and tokens that look realistic (e.g., 'YOUR_BOT_TOKEN', 'sk-xxx'). While these are obviously placeholders, some of the config examples show full JSON structures that could be copy-pasted by users who forget to replace all the placeholder values. Consider using more obviously fake values like 'REPLACE_ME_WITH_YOUR_TOKEN'.

  4. No PR description or linked issue: The PR body is empty, making it hard to understand the motivation.

Please at minimum rename the file to PicoClaw-guide.md and consider moving it to docs/.

@firaseth
Copy link
Copy Markdown
Author

firaseth commented Mar 4, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: channel type: documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants