Skip to content

Training may07#888

Open
robin-stefani wants to merge 7 commits into
devfrom
training-may07
Open

Training may07#888
robin-stefani wants to merge 7 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

Documentation PR Review

Editorial Review

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

  • Consistency — Line 14: "In this learning path, you will learn how to use Netwrix Access Analyzer 26." The parallel access-analyzer.md was updated in this same PR to use the more direct present-tense form ("This learning path introduces you to…"). Consider aligning for consistency across the learn-about paths. Suggested fix: "This learning path teaches you how to use Netwrix Access Analyzer 26."

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

  • No issues found. The shift from "you will be introduced to" to "This learning path introduces you to" is a clean passive-to-active 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

  • Consistency — Line 14: Same wording as the learn-about version — "In this learning path, you will learn how to use Netwrix Access Analyzer 26." For consistency with the rewording applied to other learning paths in this PR, consider: "This learning path teaches you how to use Netwrix Access Analyzer 26."

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

  • No issues found.

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

  • No issues found.

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. (Note: the file ends without a trailing newline — minor, but worth tidying in a future commit.)

docs/customer/training/product/index.md

  • No issues found.

docs/partner/implementation/access-analyzer-26.md

  • No issues found. Content mirrors the deleted access-analyzer-26+.md with the version-naming update and the addition of the new "1020 – Valuable Features" course.

docs/partner/implementation/access-analyzer.md

  • No issues found.

docs/partner/implementation/identity-manager.md

  • No issues found. The rewording from "certification isn't available at this time" to "certification isn't yet available" is a clean improvement. (Note: the file ends without a trailing newline.)

docs/partner/implementation/index.md

  • No issues found.

docs/partner/presales/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.

Summary

2 editorial suggestions across 23 files — both are the same consistency point: the two access-analyzer-data.md files (learn-about and product) still use the "In this learning path, you will learn how to use…" phrasing, while the parallel access-analyzer.md was updated in this PR to a more direct present-tense form. Aligning these would tighten voice across the training landing pages. Everything else is clean — version renames, time-format updates, and new-course additions are well-structured.

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

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.

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