Skip to content

Training apr28#823

Merged
jth-nw merged 3 commits intodevfrom
training-apr28
Apr 29, 2026
Merged

Training apr28#823
jth-nw merged 3 commits intodevfrom
training-apr28

Conversation

@robin-stefani
Copy link
Copy Markdown
Collaborator

Added new NAA26+ course and created customer LPs.

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 1 skipped across 8 files

Skipped (needs manual review) Reason

| docs/customer/training/product/index.md:9 — Dale: passive-voice | Actor is unknown; active-voice rewrite would require knowing which system performs enrollment |

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

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

  • Clarity — Line 16: "you will learn" uses future tense. Netwrix standards require present tense throughout. Suggested fix: "In this learning path, you learn how to use Netwrix Access Analyzer 26+."

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

No issues found.

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

  • Clarity — Line 16: Same future-tense issue as the learn-about counterpart. Suggested fix: "In this learning path, you learn how to use Netwrix Access Analyzer 26+."

docs/customer/training/product/index.md

No issues found.

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

  • Structure — Line 18: A bare bullet * 2020 <NAA /> 26+ – Concepts & Architecture is inserted directly after <NAA26Additional /> with no blank line before it, and <NAA26Concepts /> is never rendered. In the customer-facing training pages, this bullet is followed by <NAA26Concepts />, which renders the course description, prerequisites, and estimated length from 2020.md. Without that component, partner readers see only the course title — no description, no prerequisites, no duration. Suggested fix: Add a blank line before the bullet and render <NAA26Concepts /> after it, consistent with the customer pages:
    <NAA26Additional />
    
    * 2020 <NAA /> 26+ – Concepts & Architecture
    
    <NAA26Concepts />
    
    <NAMAdditional />
    

docs/partner/presales/access-analyzer-26+.md

  • Structure — Line 18: Same issue as the implementation file — <NAA26Concepts /> is imported but not rendered, and the bare bullet has no blank line separating it from <NAA26Additional />. Suggested fix: same as above.

docs/partner/presales/index.md

No issues found.

docs/partner/sales/index.md

No issues found.

src/training/access-analyzer-26+/2020.md

  • Completeness — Line 7: The course description ("provides an understanding of the application concepts and architecture") is too vague to set reader expectations. It restates the course title without explaining what topics are covered or what the learner will be able to do after completing it. Suggested fix: Add one or two specific outcomes, for example: "The Netwrix Access Analyzer 26+ – Concepts & Architecture course covers the product's data architecture, collection framework, and reporting capabilities."

Summary

5 editorial suggestions across 5 files. 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.

@robin-stefani robin-stefani marked this pull request as ready for review April 28, 2026 19:10
@robin-stefani robin-stefani requested a review from a team as a code owner April 28, 2026 19:10
@jth-nw jth-nw merged commit a4a935d into dev Apr 29, 2026
10 checks passed
jth-nw added a commit that referenced this pull request Apr 29, 2026
Merge pull request #823 from netwrix/training-apr28

Training apr28
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