Skip to content

Fix outdated Temporal Cloud UI links - #5465

Open
codemonkeycxy wants to merge 2 commits into
mainfrom
xinyi-fix-cloud-identities-doc-links-20261010
Open

codemonkeycxy wants to merge 2 commits into
mainfrom
xinyi-fix-cloud-identities-doc-links-20261010

Conversation

@codemonkeycxy

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Update the User Groups and Service Accounts docs to link to the current Identities views.
  • Update API Keys documentation to use the current /keys route and navigation label.
  • Replace all 9 references to the broken /settings/identities route and all 4 legacy /settings/api-keys links found in the documentation repo.

Notes to reviewers

  • Verified /settings/identities returns a 404. The Groups and Service Accounts links open their respective Identities tabs.
  • /settings/api-keys currently redirects to /keys; these docs now link directly to the canonical route.
  • Documentation-only change. No tests run.

@codemonkeycxy
codemonkeycxy requested a review from a team as a code owner October 11, 2026 04:09
Copilot AI balanced review requested due to automatic review settings October 11, 2026 04:09
@vercel

vercel Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
temporal-documentation Ready Ready Preview Oct 11, 2026 4:11am UTC

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

One nearby API Keys reference still uses the outdated “Settings page” terminology.

1 open finding
What changed in this PR

Updates Temporal Cloud documentation to use current identity and API key routes.

Changes:

  • Replaces broken Identities links with tab-specific routes.
  • Replaces legacy API Keys links with /keys.
  • Updates related navigation labels.
File Description
docs/​cloud/​manage-access/​user-groups.mdx Links directly to the Groups identities tab.
docs/​cloud/​manage-access/​service-accounts.mdx Links directly to the Service Accounts identities tab.
docs/​cloud/​get-started/​api-keys.mdx Uses the canonical API Keys route and label.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/cloud/get-started/api-keys.mdx Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

All targeted legacy routes were replaced consistently, with no remaining collateral references found.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced

This branch was successfully deployed

1 active deployment
Preview — d45e1db2 Deployed Oct 11, 2026 by vercel[bot]
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