Skip to content

Training may07#888

Merged
hilram7 merged 8 commits into
devfrom
training-may07
May 13, 2026
Merged

Training may07#888
hilram7 merged 8 commits into
devfrom
training-may07

Conversation

@robin-stefani
Copy link
Copy Markdown
Collaborator

Added new LTTs for NEP and NTM.
Updated 3005 NAA course and reformatted NAA estimated lengths.
Added new 3721 NIM course and updated LPs.
Added new 1020 NAA26 course, updated LPs, and renamed files and existing course to remove + from product name.

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

2 issues fixed, 2 skipped across 23 files

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

| docs/customer/training/product/index.md:9 — Dale: passive-voice | 'You will be automatically enrolled' is passive, but the agent is ambiguous (Learning Library, Netwrix, the platform) and any active rewrite would shift the subject away from 'you', changing the focus. |
| docs/partner/implementation/access-analyzer-26.md:1 — Dale: undefined-acronyms | File does not exist (access-analyzer-26+.md was renamed); no body to check. |

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

@hilram7 hilram7 mentioned this pull request May 11, 2026
hilram7
hilram7 previously approved these changes May 11, 2026
@robin-stefani robin-stefani marked this pull request as ready for review May 12, 2026 18:48
@robin-stefani robin-stefani requested a review from a team as a code owner May 12, 2026 18:48
@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

4 issues fixed, 1 skipped across 23 files

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

| docs/partner/implementation/identity-manager.md:15 — Dale: wordiness | Sentence 'When the final courses are available to grant certification, this learning path will include them' is slightly wordy but rewording risks changing nuance about course/certification dependency; left as-is. |

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

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 0 skipped across 23 files

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

This PR is primarily a renaming pass (Access Analyzer "26+" → "26"), estimated-length format normalization, and the addition of two new training courses. Most prose edits are small improvements (active voice, tighter phrasing) and read cleanly. The substantive new prose lives in the imported course files under src/training/, where most editorial concerns surface.

docs/customer/training/learn-about/access-analyzer-data.md

  • No issues found.

docs/customer/training/learn-about/access-analyzer.md

  • No issues found. The change from "you will be introduced to" to "This learning path introduces you to" is a solid active-voice improvement.

docs/customer/training/learn-about/identity-manager.md

  • No issues found.

docs/customer/training/learn-about/index.md

  • No issues found.

docs/customer/training/product/access-analyzer-data.md

  • No issues found.

docs/customer/training/product/access-analyzer/active-directory.md

  • No issues found.

docs/customer/training/product/access-analyzer/activity-monitor.md

  • No issues found.

docs/customer/training/product/access-analyzer/core.md

  • Clarity — Line 12: "It covers the core content of this application, the Netwrix Access Intelligence Console, and the Sensitive Data Add-on and applies to all solutions." The trailing "and applies to all solutions" is grammatically tied to the list and can confuse the reader on a first pass. Suggested fix: split into two sentences — "It covers the core content of this application, the Netwrix Access Intelligence Console, and the Sensitive Data Add-on. The course applies to all solutions."

docs/customer/training/product/access-analyzer/exchange.md

  • Clarity — Line 12: Same construction as core.md ("...and the Sensitive Data Add-on and applies to all solutions."). Suggested fix: split into two sentences as above.

docs/customer/training/product/access-analyzer/file-system.md

  • No issues found.

docs/customer/training/product/access-analyzer/sharepoint.md

  • No issues found.

docs/customer/training/product/access-analyzer/windows.md

  • No issues found.

docs/customer/training/product/identity-manager.md

  • No issues found.

docs/customer/training/product/index.md

  • No issues found. The rewrite of "You will be automatically enrolled..." to "The Learning Library automatically enrolls you..." is a good active-voice fix.

docs/partner/implementation/access-analyzer-26.md (new file, replacing access-analyzer-26+.md)

  • No issues found.

docs/partner/implementation/access-analyzer.md

  • No issues found.

docs/partner/implementation/identity-manager.md

  • No issues found. The "isn't yet available" rewrite is cleaner than "isn't available at this time".

docs/partner/implementation/index.md

  • No issues found.

docs/partner/presales/access-analyzer-26.md (renamed from access-analyzer-26+.md)

  • No issues found.

docs/partner/presales/access-analyzer.md

  • No issues found.

docs/partner/presales/identity-manager.md

  • No issues found.

docs/partner/presales/index.md

  • No issues found.

Imported course content (rendered into the docs pages above)

These files are not in docs/ but are imported into the customer/partner training pages and define the prose readers actually see. Editorial concerns below.

src/training/access-analyzer-26/1020.md (new file)

  • Professionalism — Line 7: The course description leans on marketing language — "comprehensive overview", "eliminate complexity, strengthen your security posture, and confidently manage today's evolving risks". The Netwrix style guide calls for a neutral, informative tone in documentation and prohibits marketing language. Suggested fix: rewrite in the same plain register used by other course descriptions (compare 2020.md: "provides an understanding of the application concepts and architecture"). For example: "The Netwrix Access Analyzer 26 – Valuable Features course covers how the product unifies data protection across data sources, automates governance, and supports controls for AI adoption. The course explains sensitive data discovery, access management, and compliance reporting, and shows how the product helps manage security posture."
  • Clarity — Line 7: "today's evolving risks" is a time-bound reference. The style guide prohibits "currently", "presently", and similar temporal hedges that go stale. Suggested fix: remove "today's" or replace with "evolving security risks".
  • Clarity — [idiom] Line 7: "regardless of where your data lives" is figurative. Suggested fix: "regardless of data location" or "across on-premises and cloud sources".
  • Consistency — Line 5: "Recommended prerequisite: none " uses lowercase "none" and has a trailing space. The sibling file 2020.md follows the convention "Recommended prerequisite: 1020 ..." and the prior 26+ version used capitalized "None". Suggested fix: "Recommended prerequisite: None" (no trailing space).

src/training/access-analyzer-26/2020.md (new file)

  • No issues found.

src/training/identity-manager/3721.md (new file)

  • Clarity — Line 7: "universes" appears in the list "operational reports, custom queries, universes, and Power BI integration" without context. The term is BI/SAP BusinessObjects jargon and won't be obvious to readers new to the product — the audience guidance in docs/CLAUDE.md says to write for "newer users who may not have deep security expertise yet" and to define terms on first use. Suggested fix: add a brief gloss — e.g., "data universes (semantic layers for BI tools)" — or restructure so the term has surrounding context.

Summary

5 editorial suggestions across 4 files (2 in docs/, 3 in imported course files). The bulk of the PR is mechanical renames and length-format updates that read cleanly. 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.

@hilram7 hilram7 merged commit eb91b8c into dev May 13, 2026
10 checks passed
DanPiazza-Netwrix pushed a commit that referenced this pull request May 13, 2026
* Added newest LTTs for NEP and NTM

* Updated NAA 3005 course, updated estimated length format for NAA course and LPs to match Netwrix University update

* added new NIM 3721 course and updated affected LPs

* Added new NAA 26 course (1020) updated affected LPs. Also renamed files to remove the + from NAA26 name.

* fix(vale): auto-fix style issues (Vale + Dale)

* fix(vale): auto-fix style issues (Vale + Dale)

---------

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
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