Skip to content

Clarify Finance Admin assignment restrictions - #5282

Open
LutaoX wants to merge 1 commit into
mainfrom
docs/clarify-finance-admin-assignment
Open

Clarify Finance Admin assignment restrictions#5282
LutaoX wants to merge 1 commit into
mainfrom
docs/clarify-finance-admin-assignment

Conversation

@LutaoX

@LutaoX LutaoX commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • State that only Account Owners can assign Finance Admin to users, groups, or Service Accounts.
  • Clarify that Global Admins can create those principals but cannot grant Finance Admin.
  • Distinguish Global Admin usage visibility from Finance Admin billing and payment access.
  • Remove the incorrect Service Account exception from the user roles page.

Validation

  • Confirmed the branch started from the latest temporalio/documentation main.
  • Confirmed git diff --check passes.
  • Local Vale was unavailable; PR CI will run the repository's Vale gate.

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@LutaoX
LutaoX requested a review from a team as a code owner September 9, 2026 15:20
@vercel

vercel Bot commented Sep 9, 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 Sep 9, 2026 3:21pm UTC

Request Review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-09T15:22:37.276749Z 649492a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 649492af8a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- Has permissions to view [billing](/cloud/billing-and-usage) information and update payment information
- Otherwise, has the same permissions as Account Read-only users
- Can be assigned to Service Accounts by a Global Admin, but otherwise can only be assigned by an Account Owner
- Only an Account Owner can assign Finance Admin to a user, group, or Service Account

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Carry the Finance Admin restriction into role-update guides

When a Global Admin follows the existing task guidance, docs/cloud/manage-access/users.mdx still says they can update any user's account-level role and identifies only Account Owner as restricted (lines 102–103). The equivalent guides in docs/cloud/manage-access/user-groups.mdx (lines 44, 87–93) and docs/cloud/manage-access/service-accounts.mdx (lines 37, 194–213) likewise tell Global Admins to select or change an account role without this exception. Update those three procedural pages so they do not direct Global Admins toward a Finance Admin assignment that the newly documented rule says they cannot perform.

AGENTS.md reference: AGENTS.md:L234-L242

Useful? React with 👍 / 👎.

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.

2 participants