Skip to content

Include instructions for setting up hierarchical team relationships f… - #38661

Open
jessiewu22 wants to merge 5 commits into
masterfrom
jessiewu22/update-teams-index
Open

Include instructions for setting up hierarchical team relationships f…#38661
jessiewu22 wants to merge 5 commits into
masterfrom
jessiewu22/update-teams-index

Conversation

@jessiewu22

Copy link
Copy Markdown
Contributor

…or new teams map feature

What does this PR do? What is the motivation?

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@jessiewu22
jessiewu22 requested a review from a team as a code owner July 31, 2026 18:59
@datadog-prod-us1-5

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@OliviaShoup OliviaShoup 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.

Lookin good! Reviewing this alongside #38660, which affects the feedback

  1. Line 76 here is a near-verbatim duplicate of line 85 in #38660 ("...visit the [Teams Map] in the Datadog Internal Developer Portal (IDP)").
  2. The same broken Teams Map URL appears in both[40] here, [7] there.
  3. #38660 removes the Teams API pointer from manage.md, while this PR adds a Teams API link here, so that pointer effectively migrates pages.

Suggestion: move these four setup methods into #38660's ## Team hierarchies section, so what, why, and how live together in one place. This page would then just cross-link, wwhich matches how it already behaves, since line 62 defers SAML detail to another page and the page closes with "To customize your team, see [Team Management][3]." That collapses most of this PR into a single pointer line and removes the duplicated sentence and duplicated URL at the same time.

If you'd rather keep the split, that's workable too. In that case the duplicated sentence on line 76 should go and this page should link to #38660's section instead.

The blocking item is the Teams Map URL, which would ship a link pinning readers to a one-hour window in the past. The list structure is the other substantive item. Everything else is line-level.

I've written the inline suggestions to work whether or not the content moves, so none of them are wasted if you consolidate.

Comment thread content/en/account_management/teams/_index.md Outdated
1. In the **Info** tab of the team page, click edit on the **Hierarchy** section
1. Select the parent or child teams associated with the team and click save

As you set up hierarchies, to explore and verify the full map of all teams in your organization, visit the [Teams Map][40] in the Datadog Internal Developer Portal (IDP).

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.

Three items here:

  • This sentence is duplicated near-verbatim in Add product description for new teams map #38660 (line 85 of manage.md). See the consolidation note in the review body. Most likely this whole section moves there and this line goes away.
  • 30 words with two stacked adverbial phrases up front ("As you set up hierarchies, to explore and verify..."). Also (IDP) is expanded and never reused on the page, so the parenthetical can go.
Suggested change
As you set up hierarchies, to explore and verify the full map of all teams in your organization, visit the [Teams Map][40] in the Datadog Internal Developer Portal (IDP).
To explore and verify the full map of all teams in your organization, see the [Teams Map][40] in the Datadog Internal Developer Portal.


If you have teams with existing members, picking the SAML strict option overrides your settings and removes team members from those teams. Picking the All Sources option preserves existing memberships. To manage teams and team membership using SAML attributes, see [Map SAML attributes to Teams][4].

## Hierarchies for Teams Map

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.

Vale flags this heading for sentence-style capitalization, but I'd settle the product name before changing anything. #38660 uses both forms in a single sentence — "Build a Teams map" (lowercase) and "[Teams Map]" (capitalized) — so the name is genuinely unsettled across these two PRs. Whichever you pick should be applied consistently in both.

Separately, worth considering a more action-oriented heading like Set up team hierarchies, which tells readers what they can do here rather than naming a concept.

Comment thread content/en/account_management/teams/_index.md Outdated
Comment thread content/en/account_management/teams/_index.md Outdated
Comment thread content/en/account_management/teams/_index.md Outdated
jessiewu22 and others added 4 commits July 31, 2026 17:19
Co-authored-by: Olivia Shoup <116908616+OliviaShoup@users.noreply.github.com>
Co-authored-by: Olivia Shoup <116908616+OliviaShoup@users.noreply.github.com>
Co-authored-by: Olivia Shoup <116908616+OliviaShoup@users.noreply.github.com>
Co-authored-by: Olivia Shoup <116908616+OliviaShoup@users.noreply.github.com>
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