Skip to content

Replace external mlflow.org links with internal paths in sidebarsClassicML.ts#18682

Merged
harupy merged 3 commits intomasterfrom
copilot/update-internal-documentation-paths
Nov 5, 2025
Merged

Replace external mlflow.org links with internal paths in sidebarsClassicML.ts#18682
harupy merged 3 commits intomasterfrom
copilot/update-internal-documentation-paths

Conversation

Copy link
Contributor

Copilot AI commented Nov 5, 2025

Converts three external documentation links to internal relative paths to enable proper navigation within self-hosted documentation deployments.

Changes:

  • https://mlflow.org/docs/latest/self-hosting/architecture/self-hosting/architecture/tracking-server/
  • https://mlflow.org/docs/latest/self-hosting/self-hosting
  • https://mlflow.org/docs/latest/self-hosting/security/basic-http-auth/self-hosting/security/basic-http-auth

All links are in the Team Collaboration sidebar section, affecting the Tracking Server, Self-Hosting, and Access Control menu items.

Original prompt

Replace external links in sidebarsClassicML.ts to use internal documentation paths for self-hosting topics.

diff --git a/docs/sidebarsClassicML.ts b/docs/sidebarsClassicML.ts
index 56ff535e51..bb4082c263 100644
--- a/docs/sidebarsClassicML.ts
+++ b/docs/sidebarsClassicML.ts
@@ -451,7 +451,7 @@ const sidebarsClassicML: SidebarsConfig = {
             {
               type: 'link',
               label: 'Tracking Server',
-              href: 'https://mlflow.org/docs/latest/self-hosting/architecture',
+              href: '/self-hosting/architecture/tracking-server/',
             },
             {
               type: 'category',
@@ -615,7 +615,7 @@ const sidebarsClassicML: SidebarsConfig = {
       items: [
         {
           type: 'link',
-          href: 'https://mlflow.org/docs/latest/self-hosting',
+          href: '/self-hosting',
           label: 'Self-Hosting',
         },
         {
@@ -625,7 +625,7 @@ const sidebarsClassicML: SidebarsConfig = {
         },
         {
           type: 'link',
-          href: 'https://mlflow.org/docs/latest/self-hosting/security/basic-http-auth',
+          href: '/self-hosting/security/basic-http-auth',
           label: 'Access Control',
         },
         {

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@harupy harupy marked this pull request as ready for review November 5, 2025 08:30
@harupy harupy added the rn/none List under Small Changes in Changelogs. label Nov 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

@Copilot Thank you for the contribution! Could you fix the following issue(s)?

⚠ Invalid PR template

This PR does not appear to have been filed using the MLflow PR template. Please copy the PR template from here and fill it out.

…lassicML.ts

Co-authored-by: harupy <17039389+harupy@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace external links with internal documentation paths Replace external mlflow.org links with internal paths in sidebarsClassicML.ts Nov 5, 2025
Copilot AI requested a review from harupy November 5, 2025 08:42
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

Documentation preview for 123d6eb is available at:

More info
  • Ignore this comment if this PR does not change the documentation.
  • The preview is updated when a new commit is pushed to this PR.
  • This comment was created by this workflow run.
  • The documentation was built by this workflow run.

@harupy harupy added the team-review Trigger a team review request label Nov 5, 2025
@harupy harupy merged commit efab7c3 into master Nov 5, 2025
46 of 48 checks passed
@harupy harupy deleted the copilot/update-internal-documentation-paths branch November 5, 2025 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rn/none List under Small Changes in Changelogs. team-review Trigger a team review request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants