Skip to content

fix(web): route enterprise documentation links - #40957

Merged
hyoban merged 5 commits into
mainfrom
codex/route-enterprise-doc-links
Aug 21, 2026
Merged

fix(web): route enterprise documentation links#40957
hyoban merged 5 commits into
mainfrom
codex/route-enterprise-doc-links

Conversation

@hyoban

@hyoban hyoban commented Aug 19, 2026

Copy link
Copy Markdown
Member

Important

This draft PR does not have an associated issue yet.

Summary

  • Route Web documentation links through an edition-aware shared helper.
  • Send Enterprise deployments to enterprise-docs.dify.ai without a version segment and normalize public documentation paths.
  • Migrate the Goto Anything docs command, permission references, the CLI tool dialog, and the New RAG knowledge switcher to the shared routing.
  • Preserve explicitly supplied documentation base URLs and leave pipeline templates unchanged.

Screenshots

Not applicable. This change affects destination URLs without changing the UI.

Testing

  • vp staged
  • pnpm --dir web exec vitest run context/i18n.spec.ts app/components/goto-anything/actions/commands/__tests__/direct-commands.spec.ts app/components/goto-anything/actions/commands/__tests__/slash.spec.tsx features/agent-v2/agent-detail/configure/components/orchestrate/tools/cli-tool/__tests__/dialog.spec.tsx features/new-rag/__tests__/new-knowledge-list.spec.tsx (95 tests passed)

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This does not apply to typos.)
  • I have added tests for the behavior changes and kept the change atomic.
  • I have updated the documentation accordingly.
  • I ran vp staged for the frontend changes.

From Codex

@github-actions github-actions Bot added the web This relates to changes on the web. label Aug 19, 2026
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.78%. Comparing base (b78a346) to head (9d42f2d).
⚠️ Report is 63 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #40957      +/-   ##
==========================================
+ Coverage   86.76%   86.78%   +0.01%     
==========================================
  Files        5199     5201       +2     
  Lines      292699   292767      +68     
  Branches    58157    58207      +50     
==========================================
+ Hits       253960   254072     +112     
+ Misses      33720    33676      -44     
  Partials     5019     5019              
Flag Coverage Δ
dify-ui 93.60% <ø> (ø)
web 86.74% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 20, 2026
@hyoban
hyoban marked this pull request as ready for review August 21, 2026 02:13
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Aug 21, 2026
@hyoban
hyoban added this pull request to the merge queue Aug 21, 2026
@dosubot dosubot Bot added the refactor label Aug 21, 2026
Merged via the queue into main with commit 3d08b2f Aug 21, 2026
43 checks passed
@hyoban
hyoban deleted the codex/route-enterprise-doc-links branch August 21, 2026 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer refactor size:L This PR changes 100-499 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants