Skip to content

docs: Update dingtalk QR code and default port for docker#681

Merged
xieyxclack merged 2 commits intomainfrom
weirui/dev/fix0305
Mar 5, 2026
Merged

docs: Update dingtalk QR code and default port for docker#681
xieyxclack merged 2 commits intomainfrom
weirui/dev/fix0305

Conversation

@rayrayraykk
Copy link
Copy Markdown
Member

@rayrayraykk rayrayraykk commented Mar 5, 2026

Description

[Describe what this PR does and why]

Related Issue: Fixes #(issue_number) or Relates to #(issue_number)

Security Considerations: [If applicable, e.g. channel auth, env/config handling]

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring

Component(s) Affected

  • Core / Backend (app, agents, config, providers, utils, local_models)
  • Console (frontend web UI)
  • Channels (DingTalk, Feishu, QQ, Discord, iMessage, etc.)
  • Skills
  • CLI
  • Documentation (website)
  • Tests
  • CI/CD
  • Scripts / Deploy

Checklist

  • I ran pre-commit run --all-files locally and it passes
  • If pre-commit auto-fixed files, I committed those changes and reran checks
  • I ran tests locally (pytest or as relevant) and they pass
  • Documentation updated (if needed)
  • Ready for review

Testing

[How to test these changes]

Local Verification Evidence

pre-commit run --all-files
# paste summary result

pytest
# paste summary result

Additional Notes

[Optional: any other context]

Summary by CodeRabbit

  • Documentation

    • Updated Docker run instructions across install, quick-start, and FAQ pages to bind to localhost (127.0.0.1) instead of all network interfaces.
    • Refreshed community pages and README badge/asset images for Discord/DingTalk.
  • Chores

    • Updated Docker build/run examples in scripts to reflect localhost binding for example commands.

Copilot AI review requested due to automatic review settings March 5, 2026 03:54
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 5, 2026

📝 Walkthrough

Walkthrough

Updates Docker run examples to bind container ports to localhost (127.0.0.1) across docs and code; updates DingTalk/Discord badge image URLs and dimensions; refines the PR-label GitHub Actions workflow with guarded first-time-contributor checks and error handling.

Changes

Cohort / File(s) Summary
Docker CLI & docs
README.md, README_ja.md, README_zh.md, scripts/docker_build.sh, website/public/docs/faq.en.md, website/public/docs/faq.zh.md, website/public/docs/quickstart.en.md, website/public/docs/quickstart.zh.md, website/src/components/QuickStart.tsx
Replaced host port bindings from -p 8088:8088 (and -p 3000:3000) to loopback bindings like -p 127.0.0.1:8088:8088 (and 127.0.0.1:3000:3000) in examples and UI component.
Community asset updates
README.md, README_ja.md, README_zh.md, website/public/docs/community.en.md, website/public/docs/community.zh.md
Swapped image URLs for the DingTalk (contact) badge; adjusted image source and dimensions (400x400 -> 458x460) in some locales.
CI workflow: PR labeling
.github/workflows/pr-label.yml
Added guarded first-time contributor check using PR search results, computes isFirstTime from searchResult.total_count === 0, logs existing closed-PR cases, and handles 422 "cannot be searched" errors by aborting labeling path.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • xieyxclack

Poem

🐰 I hopped through docs at night,
Ports snug on localhost tight,
Badges swapped their bright attire,
Workflows whisper a safer wire,
I munch on carrots, code feels right. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title 'docs: Update dingtalk QR code' accurately reflects the primary change, but significantly understates the scope. The changeset includes 10+ documentation files modified with Docker binding changes (exposing only localhost instead of all interfaces) and DingTalk QR code updates. The title mentions only the QR code update while Docker changes span most of the modified files. Revise title to reflect the main changes: 'docs: Update DingTalk QR code and restrict Docker to localhost binding' or similar, to accurately represent both major changes in the changeset.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch weirui/dev/fix0305

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates website/docs content by (1) refreshing the DingTalk community QR code image and (2) changing Docker run examples to bind the exposed port to localhost (127.0.0.1) for safer defaults when running locally.

Changes:

  • Update Docker run commands in Quick Start/FAQ/README/docs to use -p 127.0.0.1:8088:8088.
  • Update DingTalk QR code image URL in community pages and READMEs.
  • Update scripts/docker_build.sh printed run instructions to match the localhost-only port binding.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
website/src/components/QuickStart.tsx Updates Docker quickstart command shown in the UI to bind port to localhost.
website/public/docs/quickstart.zh.md Updates Docker quickstart command to bind port to localhost.
website/public/docs/quickstart.en.md Updates Docker quickstart command to bind port to localhost.
website/public/docs/faq.zh.md Updates Docker commands in FAQ to bind port to localhost.
website/public/docs/faq.en.md Updates Docker commands in FAQ to bind port to localhost.
website/public/docs/community.zh.md Updates DingTalk QR image URL in community table.
website/public/docs/community.en.md Updates DingTalk QR image URL in community table.
scripts/docker_build.sh Updates printed docker run examples to bind ports to localhost.
README_zh.md Updates Docker command and DingTalk QR image URL.
README_ja.md Updates Docker command and DingTalk QR image URL.
README.md Updates Docker command and DingTalk QR image URL.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rayrayraykk rayrayraykk changed the title docs: Update dingtalk QR code docs: Update dingtalk QR code and default port for docker Mar 5, 2026
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/pr-label.yml:
- Around line 22-28: The search-based first-time check using
github.rest.search.issuesAndPullRequests is unreliable; replace it by listing
repo PRs with the pulls API and checking the PR author directly. Call
github.rest.pulls.list({ owner: context.repo.owner, repo: context.repo.repo,
state: 'all', per_page: 100 }) (with pagination if needed), then compute
isFirstTime by ensuring no returned PR has pr.user.login === author (or count
matching PRs); update the code paths that set isFirstTime and the logging (the
variables/functions to change are isFirstTime and the block using
github.rest.search.issuesAndPullRequests).

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22a6b909-fc98-49df-8127-b5e224cfcbd8

📥 Commits

Reviewing files that changed from the base of the PR and between 81f2654 and 555f944.

📒 Files selected for processing (1)
  • .github/workflows/pr-label.yml

@xieyxclack xieyxclack merged commit 011715a into main Mar 5, 2026
7 checks passed
@rayrayraykk rayrayraykk deleted the weirui/dev/fix0305 branch March 6, 2026 13:41
tudan110 pushed a commit to tudan110/QwenPaw that referenced this pull request Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants