You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Completeness — Line 113: The revised version note no longer identifies which NPS-D build the screenshot depicts. The previous wording told readers the image showed 26.06; now the image shows 26.09.0 but nothing says so. A reader running an earlier build sees an Auto-Redirect to IdP row in the screenshot that isn't in their UI, with no explanation. Suggested fix: "Version note: This screenshot shows NPS-D 26.09.0. Starting with NPS-D 26.09.0, the SAML/SSO Configuration section includes an Auto-Redirect to IdP row with the redirect to IdP on login page checkbox."
Clarity — Line 113: The note names the UI location as the SAML/SSO Configuration section, but step 1 of this procedure (line 90) sends the reader to Configure > Server > SAML Configuration. A reader can't tell whether these are the same screen or two different ones. Suggested fix: use one label consistently — either match the procedure with "the SAML Configuration page", or add the mapping once (for example, "the SAML Configuration page (SAML/SSO Configuration section)").
Structure — Line 113: The version note uses a Markdown blockquote, while the rest of the doc set uses Docusaurus admonitions for supplementary information. Since this line is already being rewritten, converting it keeps the callout styling consistent with other Netwrix pages. Suggested fix: replace the blockquote with :::note / ::: around the version note text.
Summary
3 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.
| docs/privilegesecurediscovery/2.22/administration/configuration/microsoftentraid/samlsso.md:150 — Dale: passive-voice | "The expected NPS-D identity is signed in." reads as a state description, and the surrounding bullets are parallel state checks ("is correct", "is on the final application route", "still works"). Any active rewrite would break that parallelism or shift the meaning. |
| docs/privilegesecurediscovery/2.22/administration/configuration/microsoftentraid/samlsso.md:117 — Dale: undefined-acronyms | IdP appears at lines 100-117 only inside literal UI control names ("Auto-Redirect to IdP", "force IdP reauthentication"), which must match the product interface verbatim. The acronym is spelled out as "identity provider (IdP)" at line 122. |
| docs/privilegesecurediscovery/2.22/administration/configuration/microsoftentraid/samlsso.md:183 — Dale: wordiness | "fails in a controlled manner" is slightly wordy, but it carries a specific acceptance-test meaning (graceful failure rather than any failure). No shorter alternative preserves that distinction. |
Ask @claude on this PR if you'd like an explanation of any fix.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
Generated with AI
Co-Authored-By: Claude Code ai@netwrix.com