Skip to content

DOCS-2724 / 26 / Docs 2724 smb audit documentation is misleading (by DjP-iX) - #4792

Open
bugclerk wants to merge 2 commits into
26from
DOCS-2724-26
Open

DOCS-2724 / 26 / Docs 2724 smb audit documentation is misleading (by DjP-iX)#4792
bugclerk wants to merge 2 commits into
26from
DOCS-2724-26

Conversation

@bugclerk

@bugclerk bugclerk commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Updates SMB auditing documentation inconsistencies identified in the ticket.

Also updates the title of the Audit Logs tutorial to use a gerund form title, matching the convention for other tutorials.

Thanks for contributing to TrueNAS documentation! By opening a Pull Request, you're acknowledging that your changes will be distributed under the Creative Commons 4.0 license.

Original PR: #4786
Jira URL: https://ixsystemsinc.atlassian.net/browse/DOCS-2724

(cherry picked from commit ed370c5)
@bugclerk
bugclerk requested a review from a team as a code owner September 9, 2026 13:07
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Vale Style Guide Issues

📊 Summary: 82 issues found in changed files - 6 errors, 0 warnings, 76 suggestions

Changed files with issues:

content/SystemSettings/Audit/Auditing.md (70 issues)

  • Line 24 - suggestion: Spell out 'USER' on first use
  • Line 26 - suggestion: Spell out 'INFO' on first use
  • Line 31 - suggestion: Spell out 'VFS' on first use
  • Lines 54, 60, 64, 69, 74, 80, 89, 109, 114, 118, 149, 167, 184, 198, 212 - suggestion: Use active voice instead of passive voice
  • Lines 81, 108, 113 - suggestion: Keep sentences under 30 words
  • Lines 95, 102 - suggestion: Spell out 'TCON' on first use
  • Lines 118, 135 - error: Use 'filesystem' instead of 'file system'
  • Lines 124, 128 - suggestion: Spell out 'DOS' on first use
  • Line 135 - suggestion: Spell out 'OWNER' on first use
  • Lines 147, 151, 152, 153, 156, 212, 216 - suggestion: Spell out 'JSON' on first use
  • Lines 151, 212 - suggestion: Spell out 'YAML' on first use
  • Line 197 - suggestion: Spell out 'CLOSE' on first use
  • Line 200 - suggestion: Spell out 'SQL' on first use
  • Line 200 - suggestion: Spell out 'JQL' on first use
  • Line 228 - error: Use a numeral plus the units.

static/includes/ConfigureSMBShareAuditingSCALE.md (12 issues)

  • Lines 6, 42, 47 - suggestion: Use active voice instead of passive voice
  • Lines 17, 39 - suggestion: Keep sentences under 30 words
  • Line 39 - error: Avoid possessive forms ('user's'). Rephrase to avoid using apostrophes for possession.
  • Line 51 - error: Use 'could' instead of 'may'

🔧 To fix: Review the issues above and update your documentation accordingly.

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