Skip to content

NDM 448122 add documentation for co-manged global setting#1243

Open
stas-kukhar-nwx wants to merge 6 commits into
devfrom
ndm/448122-add-documentation-for-co-manged-global-setting
Open

NDM 448122 add documentation for co-manged global setting#1243
stas-kukhar-nwx wants to merge 6 commits into
devfrom
ndm/448122-add-documentation-for-co-manged-global-setting

Conversation

@stas-kukhar-nwx

@stas-kukhar-nwx stas-kukhar-nwx commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Adds section that describes new global setting for displaying the co-managed groups in portal
https://dev.azure.com/NetwrixCorporation/Netwrix%20GroupID/_workitems/edit/448122

@github-actions

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

17 issues fixed, 3 skipped across 1 files

Category Fixes
Contractions 3
Substitutions 2
AllowsYouTo (rewrite) 1
NoteThat (rewrite) 1
OxfordComma (rewrite) 1
PlainTextAdmonition (rewrite) 1
TypeVsEnter (rewrite) 1
Dale: passive-voice 6
Dale: wordiness 1
Skipped (needs manual review) Reason

| docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md:37 — Dale: passive-voice | 'the displayName attribute is specified, indicating that ... are sorted' — rewriting both passives risks altering the meaning of the default-value description in a dense reference table |
| docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md:49 — Dale: passive-voice | 'When the toggle button is disabled / When enabled' describe a UI state; active rewrites would shift meaning toward a user action |
| docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md:68 — Dale: passive-voice | 'By default, the setting is disabled' describes a state, not an action; active rewrite would change meaning |

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md

  • Clarity — Line 35: This row's inline note was changed to **Note:** (bold), but the sibling rows in the same Portal & Search table still use the unformatted NOTE: prefix (see the Search Default, Toolbar Default Most Recent Used Object Count, and Default Search Page Size rows). Mixing the two styles inside one table reads as inconsistent. Suggested fix: either revert this cell to NOTE: to match its neighbors, or apply **Note:** uniformly to every note prefix in the table.
  • Completeness — Line 43: The new Display Co-Managed Groups in My Groups row references a per-user Display Exchange co-managed groups preference "from the Settings panel in the portal," but this row is placed above the section note (line 60) that already states individual users can personalize all Listings Display settings except Display Nested Ownership. The row's extra clause about the per-user preference is therefore partly redundant with that note. Consider trimming the redundant explanation so the admin-facing description stays focused on the portal-wide default, and let the existing section note cover per-user personalization. (Optional — the current text is accurate, just slightly repetitive.)
  • Structure — Line 43: Placement of the new row is correct — it sits logically between Display Groups in My Groups and the other Display Groups in My ... rows, grouping the ownership-visibility settings together. No change needed.

Summary

2 editorial suggestions (1 minor, 1 optional) across 1 file. The remaining changes are voice, tense, and word-choice edits handled by the Vale/Dale auto-fix workflow. Vale and Dale issues are auto-fixed separately.


What to do next:

Comment @claude on this PR followed by your instructions to get help:

  • @claude fix all issues — fix all editorial issues
  • @claude help improve the flow of this document — get writing assistance
  • @claude explain the voice issues — understand why something was flagged

You can ask Claude anything about the review or about Netwrix writing standards.

Automated fixes are only available for branches in this repository, not forks.

@github-actions

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

1 issues fixed, 2 skipped across 1 files

Category Fixes
Dale: passive-voice 1
Skipped (needs manual review) Reason

| docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md:37 — Dale: passive-voice | 'the displayName attribute is specified, indicating that listings and search results are sorted' — rewriting to active would require inventing/altering the doer and risks changing meaning in a dense table cell. |
| docs/directorymanager/11.1/admincenter/applications/portal/server/advanced.md:49 — Dale: passive-voice | 'When the toggle button is disabled/enabled' describes UI state (adjectival), not an action needing an agent; pervasive throughout and changing it would not improve clarity. |

Ask @claude on this PR if you'd like an explanation of any fix.

@stas-kukhar-nwx
stas-kukhar-nwx marked this pull request as ready for review July 22, 2026 08:40
@stas-kukhar-nwx
stas-kukhar-nwx requested a review from a team as a code owner July 22, 2026 08:40
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