Skip to content

docs: fix #1001 — ndm adv 2025 014 broken link#1002

Open
jth-nw wants to merge 4 commits into
devfrom
fix/issue-1001-ndm-adv-2025-014-broken-link
Open

docs: fix #1001 — ndm adv 2025 014 broken link#1002
jth-nw wants to merge 4 commits into
devfrom
fix/issue-1001-ndm-adv-2025-014-broken-link

Conversation

@jth-nw

@jth-nw jth-nw commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Closes #1001

What changed

fix/issue-1001-ndm-adv-2025-014-broken-link

Update NDM11-ADV-2025-014-Utility download URL from
/products/directorymanager/11.1/ to /products/groupid/11.1/ as
reported in the issue.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

22 issues fixed, 3 skipped across 1 files

Category Fixes
AllowsYouTo (rewrite) 1
FollowTheStepsTo (rewrite) 3
FormalHedging (rewrite) 1
OnceUsage (rewrite) 1
Dale: passive-voice 10
Dale: positional-references 1
Dale: wordiness 5
Skipped (needs manual review) Reason

| docs/directorymanager/11.1/install/securityutility.md:18 — Dale: wordiness | 'Click [here]' link text is a Vale WeakLinkText concern handled by the vale-autofix workflow, not Dale |
| docs/directorymanager/11.1/install/securityutility.md:49 — Dale: misplaced-modifiers | Bullet 'version — Displays the NDM11-ADV-2025-014-Utility version' may be missing dashes ('--version'), but fix would change technical meaning |
| docs/directorymanager/11.1/install/securityutility.md:76 — Dale: passive-voice | 'this connection is required' uses 'required' as an adjective; rewriting risks changing meaning |

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/directorymanager/11.1/install/securityutility.md

This PR is a strong voice-and-clarity pass — most changes convert passive constructions to active voice, replace wordy phrases ("In the event of", "On doing so", "Next, run"), and turn "Follow the steps to…" openers into direct "To…, complete these steps:" leads. These are all improvements. One structural item to consider:

  • Structure — Line 34: In the numbered list under "Restrict IP Addresses," items 1–3 now lead with a verb describing what the utility does ("Defines…", "Detects…", "Supports adding…"), but item 4 breaks that parallelism by switching the subject to the IP rule: "The IP rule applies to the following services/virtual directories:". Suggested fix: keep the utility as the subject to match the surrounding items — "Applies the IP rule to the following services/virtual directories:".

Everything else in the diff reads cleanly. No clarity or completeness gaps introduced by the added lines.

Summary

1 editorial suggestion across 1 file. 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

1 issues fixed, 3 skipped across 1 files

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

| docs/directorymanager/11.1/install/securityutility.md:125 — Dale: positional-references | 'password copied from above' appears inside a command-example placeholder, not body prose; rewriting would alter the command syntax shown to the reader. |
| docs/directorymanager/11.1/install/securityutility.md:44 — Dale: wordiness | 'This will present the following options' uses future tense/vague pronoun, but no Dale rule covers tense and a rewrite risks changing meaning. |
| docs/directorymanager/11.1/install/securityutility.md:77 — Dale: passive-voice | 'this connection is required' is a stative/parenthetical clause; recasting to active could distort the intended meaning. |

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.

Fix: Broken Link for [ADV-2025-014] in the NDM Doc

1 participant