Skip to content

EPP small Changes to be committed:#753

Merged
bturlea merged 5 commits intodevfrom
epp_minorupdates
Apr 15, 2026
Merged

EPP small Changes to be committed:#753
bturlea merged 5 commits intodevfrom
epp_minorupdates

Conversation

@krzysztofstaszalek
Copy link
Copy Markdown
Collaborator

EPP modified:   docs/endpointprotector/admin/dc_module/globalrights.md

	modified:   docs/endpointprotector/admin/dc_module/globalrights.md
@github-actions
Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/endpointprotector/admin/dc_module/globalrights.md

  • Clarity — Line 57 vs. line 60: The list item uses "Audio cards" while the note immediately below uses "sound cards." These appear to refer to the same device type, but the inconsistency could make readers wonder if they are different things. Suggested fix: use the same term in both places — "Audio cards" is the more specific, hardware-oriented term already established in the list; change the note to read "Audio card control is supported only on Windows..."

  • Clarity — Line 60: "Audio Device Control" is not defined or linked anywhere in this document. It reads like a formal feature name, but without context, readers can't tell whether it refers to a specific product module or is a generic description. Suggested fix: either lowercase it ("Audio device control") if it's a generic description, or introduce it as a named feature with a cross-reference if it's a formal module name (consistent with how "Trusted Device™" is handled elsewhere on the page).

  • Completeness — Lines 59–61: The note tells readers what audio card control does not support (input channels, non-Windows platforms) but says nothing about what a reader can actually do with the audio card right. After reading this entry, a user doesn't know whether they can allow/deny audio output access using the standard rights workflow, or whether additional setup is required. Suggested fix: add a sentence stating what the supported capability is before describing its limits — for example: "Audio card control allows you to allow or deny audio output access on Windows endpoints. Audio input channels, whether analog or digital, are not covered."

Summary

3 editorial suggestions across 1 file. 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

17 issues fixed, 0 skipped across 1 files

Category Fixes
Contractions 5
OxfordComma (rewrite) 3
Dale: passive-voice 7
Dale: positional-references 1
Dale: wordiness 1

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

@bturlea bturlea merged commit 1fff04c into dev Apr 15, 2026
10 checks passed
@krzysztofstaszalek krzysztofstaszalek deleted the epp_minorupdates branch April 15, 2026 07:56
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