Skip to content

Fix Logon Activity permissions doc for Auditor 10.8#1021

Open
pavelshabanov2025 wants to merge 1 commit into
devfrom
pt4/pshabanov/nla_permis_PolAdtEv
Open

Fix Logon Activity permissions doc for Auditor 10.8#1021
pavelshabanov2025 wants to merge 1 commit into
devfrom
pt4/pshabanov/nla_permis_PolAdtEv

Conversation

@pavelshabanov2025
Copy link
Copy Markdown
Contributor

@pavelshabanov2025 pavelshabanov2025 commented May 27, 2026

  • Replace positional reference "(see below)" with named anchor link
  • Remove "assignment" to fix wordiness in minimum rights description
  • Rewrite passive "compression disabled" to active voice
  • Tighten introductory sentence in configuration section
  • Remove HKEY_LOCAL_MACHINE\SECURITY\Policy\PolAdtEv registry key entry
  • Remove prerequisite paragraph about granting Read on SECURITY registry key

Co-Authored-By: Claude claude-sonnet-4-6 noreply@anthropic.com

@pavelshabanov2025 @claude
Fix style issues in Logon Activity permissions doc (10.8)
735db12 
- Replace positional reference "(see below)" with named anchor link
- Remove "assignment" to fix wordiness in minimum rights description
- Rewrite passive "compression disabled" to active voice
- Tighten introductory sentence in configuration section
- Remove HKEY_LOCAL_MACHINE\SECURITY\Policy\PolAdtEv registry key entry
- Remove prerequisite paragraph about granting Read on SECURITY registry key

Generated with AI

Co-Authored-By: Claude claude-sonnet-4-6 <noreply@anthropic.com>
Co-Authored-By: Claude Code <ai@netwrix.com>
@pavelshabanov2025 pavelshabanov2025 requested a review from a team as a code owner May 27, 2026 09:05
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

Documentation PR Review

Editorial Review

docs/auditor/10.8/configuration/logonactivity/permissions/permissions.md

  • Clarity — Line 23: "automatically adjust audit settings" is abstract — the reader doesn't yet know which audit settings are being referenced or what "adjust" means in the monitoring plan workflow. Suggested fix: "...and don't want Auditor to automatically adjust audit policy settings on monitored systems." (or similar wording that names the specific settings the option controls).

Summary

1 editorial suggestion 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

github-actions Bot commented May 27, 2026

Auto-Fix Summary

0 issues fixed, 2 skipped across 1 files

Skipped (needs manual review) Reason

| docs/auditor/10.8/configuration/logonactivity/permissions/permissions.md:14 — Dale: passive-voice | 'is enabled/disabled' describes a feature's configuration state, not an action; active rewrite (e.g., 'If you enable...') changes meaning since the reader may not be the agent who enabled it |
| docs/auditor/10.8/configuration/logonactivity/permissions/permissions.md:15 — Dale: passive-voice | Same as line 14; 'configured with minimum rights' is an adjectival past-participle phrase modifying 'account' and reads naturally |

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrey-isakov-nwx andrey-isakov-nwx Awaiting requested review from andrey-isakov-nwx

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@pavelshabanov2025