Skip to content

Auditor misprints#390

Open
konstantin-melnikov-docs wants to merge 7 commits intodevfrom
auditor-misprints
Open

Auditor misprints#390
konstantin-melnikov-docs wants to merge 7 commits intodevfrom
auditor-misprints

Conversation

@konstantin-melnikov-docs
Copy link
Contributor

@konstantin-melnikov-docs konstantin-melnikov-docs commented Mar 2, 2026

Fix documentation quality issues across Auditor 10.8 and KB Auditor including spelling errors, typos, and formatting corrections.
Add missing Azure Files data source reference to configuration overview and resolve Unicode character issues in related product documentation.

konstantin-melnikov-docs and others added 3 commits March 2, 2026 21:18
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/auditor/10.8/admin/monitoringplans/oracle/overview.md

Line 84: "audit database" should be "Audit Database" (capitalized) to match the standard capitalization used throughout the documentation when referring to the Netwrix Auditor Audit Database.

Current:

Netwrix Auditor Server then writes the Activity Records to the audit database (default retention

Should be:

Netwrix Auditor Server then writes the Activity Records to the Audit Database (default retention

To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

Corrected spelling errors across KB articles and versioned docs:
- Secutre → OneSecure (9 occurrences)
- Сonnection → Connection (8, Cyrillic C)
- contoller → controller (4)
- recepient → recipient (4)
- attribites → attributes (3)
- decomissioned → decommissioned (3)
- Incorrecty → Incorrectly (3)
- timout → timeout (3)
- Plus 15 additional single-occurrence fixes

Updated sidebar labels and titles in 4 KB articles.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/kb/auditor/event-id-reference/health-log-events/event-id-1223-in-health-log.md

Line 30: Placeholder formatting. Uses %recipient% with percent signs. Replace with angle brackets: <recipient>.

docs/kb/auditor/event-id-reference/health-log-events/event-id-1260-1266-in-health-log.md

Line 18: Placeholder formatting. Uses %affected_SharePoint_farm% and %error% with percent signs. Replace with angle brackets: <affected_SharePoint_farm> and <error>.

docs/kb/auditor/event-id-reference/health-log-events/event-id-1267-1273-in-health-log.md

Line 18: Placeholder formatting. Uses %affected_SharePoint_farm% with percent signs. Replace with angle brackets: <affected_SharePoint_farm>.

docs/kb/auditor/troubleshooting-and-errors/data-collection-errors/ts-collection-failed-warning.md

Line 23: Placeholder formatting. Uses %domain controller name% with percent signs. Replace with angle brackets: <domain controller name>.

Line 24: Placeholder formatting. Uses %domain controller name% with percent signs. Replace with angle brackets: <domain controller name>.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

None.

All changes in this PR are simple typo and spelling corrections that improve documentation quality. The corrections include:

  • Spelling fixes: "conjuction" → "conjunction", "applicaton" → "application", "deafult" → "default"
  • Product name corrections: "ConectWise" → "ConnectWise", "Shapoint" → "SharePoint", "OneSecutre" → "OneSecure"
  • Placeholder fixes: "%recepient%" → "%recipient%", "%domain contoller name%" → "%domain controller name%"
  • Cyrillic character fixes: "Сonnection" → "Connection" (replacing Cyrillic 'C' with Latin 'C')
  • Title/heading corrections: "Incorrecty" → "Incorrectly", "decomissioned" → "decommissioned"

All corrections align with the Netwrix Style Guide and documentation standards. No new style guide violations were introduced.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/auditor/10.8/addon/connectwise/operationalsettings.md

Line 29: Contains "i.e." (twice) which should be written as "that is" or restructured for plain English per style guide preference. The line has "The default values is 900 seconds, i.e., 15 minutes" and "I.e., this option defines the maximum delay".

docs/auditor/10.8/admin/reports/types/stateintime/activedirectory.md

Line 48: Uses "Note that unselected attributes have no values." This should be converted to a Docusaurus admonition block per the NoteThat rule:

:::note
Unselected attributes have no values.
:::

docs/auditor/10.8/tools/passwordexpirationnotifier/overview.md

Line 63: Contains "In order to send a test email" which should be simplified to "To send a test email" per conciseness guidelines.

Line 64: Contains "In order to send a test email" which should be simplified to "To send a test email" per conciseness guidelines.

Line 65: Contains "In order to send a test email" which should be simplified to "To send a test email" per conciseness guidelines.

docs/auditor/10.8/tools/passwordexpirationnotifier/ports.md

Line 11: Uses inline "NOTE:" instead of a Docusaurus admonition block. Should be converted to:

:::note
Tip for reading the table – On the computer where the Netwrix Auditor server resides (source), allow outbound connections to remote 389 the TCP port. On domain controllers in your domain (target), allow inbound connections to the local 389 TCP port.
:::

docs/kb/auditor/features-and-operations/glossaries-and-faqs/incorrecty-display-names-in-the-what-changed-column.md

Line 18: "Incorrectly Display Names" is grammatically incorrect. Should be "Incorrect Display Names" or "Incorrectly Displayed Names".

Line 20: Same issue - "Incorrectly Display Names" should be "Incorrect Display Names" or "Incorrectly Displayed Names".

Line 24: Same issue - "Incorrectly Display Names" should be "Incorrect Display Names" or "Incorrectly Displayed Names".


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/auditor/10.8/addon/connectwise/operationalsettings.md

Line 29: Subject-verb agreement error. "The default values is 900 seconds" should be "The default value is 900 seconds" (or "values are" if plural is intended).

docs/auditor/10.8/addon/cyberark/overview.md

Line 45: Latin abbreviation "etc." used. Per style guide, use "and more" or list all items instead of "etc."

docs/auditor/10.8/admin/monitoringplans/exchange/scope.md

Line 60: Incorrect article usage. "To exclude the certain user's mailbox" and "To exclude the certain folder" should use "a certain" instead of "the certain".


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

None.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

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