Skip to content

docs: fix #1059 — ndm signing key download links#1060

Open
jth-nw wants to merge 2 commits into
devfrom
fix/issue-1059-ndm-signing-key-download-links
Open

docs: fix #1059 — ndm signing key download links#1060
jth-nw wants to merge 2 commits into
devfrom
fix/issue-1059-ndm-signing-key-download-links

Conversation

@jth-nw

@jth-nw jth-nw commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Closes #1059

What changed

fix/issue-1059-ndm-signing-key-download-links

@github-actions @claude
docs: fix #1059 — update broken Signing Key Utility download links
073d50c 
Replace directorymanager path with groupid path in download URLs per
the correct links from security advisory ADV-2025-004.

Co-Authored-By: Claude <noreply@anthropic.com>
@jth-nw jth-nw requested a review from a team as a code owner June 3, 2026 07:27
@jth-nw jth-nw mentioned this pull request Jun 3, 2026
Open
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/directorymanager/11.1/admincenter/service/securityservice/signkeyutility.md

  • No issues found. The change updates two download URLs from the legacy directorymanager path to the groupid path. The link text and link target match, the surrounding context (which describes the product as "Netwrix Directory Manager (formerly GroupID)") makes the groupid URL segment unsurprising to the reader, and the bulleted version labels remain accurate.

Summary

0 editorial suggestions 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

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

13 issues fixed, 2 skipped across 1 files

Category Fixes
FollowTheStepsTo (rewrite) 1
TypeVsEnter (rewrite) 3
Dale: misplaced-modifiers 1
Dale: passive-voice 7
Dale: wordiness 1
Skipped (needs manual review) Reason

| docs/directorymanager/11.1/admincenter/service/securityservice/signkeyutility.md:87 — Dale: passive-voice | 'Use the same parameters given in the Generate a SigningKey section' — 'given' is a participial adjective; rewriting risks changing meaning |
| docs/directorymanager/11.1/admincenter/service/securityservice/signkeyutility.md:91 — Dale: passive-voice | 'newly generated signing key' — participial adjective modifying noun, acceptable usage |

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

Reviewers

@dave-miles65 dave-miles65 Awaiting requested review from dave-miles65

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix: NDM Installation doc has a broken link

1 participant

@jth-nw