add Password Reset link missing with Duo and PPE Client#914
Merged
Conversation
…icle Generated with AI Co-Authored-By: Claude Code <ai@netwrix.com>
…g-duo-and-ppe-client - Fix products field: password-reset → passwordreset - Add required kb tag - Fix title case: "installed" → "Installed" in frontmatter title - Fix sidebar_label to include "with" and align with title - Format bare Duo URL as a proper markdown link (Dale)
…ssing-duo-and-ppe-client - Expand Cause section with context on credential stack ordering and ProvidersWhitelist - Restructure Resolution steps with ordered uninstall/reinstall sub-bullets - Fix step numbering and NOTE placement - Fix subject-verb agreement in Cause
… callout docs/kb/passwordreset/portal-access-and-authentication/password-reset-link-missing-duo-and-ppe-client.md - Split Cause section into two paragraphs for readability - Moved IMPORTANT callout above reinstall bullet list in step 3 - Moved registry key block inside the IMPORTANT callout at end of article - Confirmed Duo instructions external link resolves correctly - Confirmed rendering in both light and dark mode
hilram7
approved these changes
May 15, 2026
Collaborator
hilram7
left a comment
There was a problem hiding this comment.
KB Review
Files reviewed: 1
Changes applied
Vale / Derek:
- Fixed
productsfield frompassword-resettopasswordreset(invalid product ID) - Added missing
kbtag totagsfield
Dale:
- Rewrote "It is imperative to ensure" as "Ensure" (impersonal construction)
Proofread:
- Fixed sequential step numbering (gap after step 3)
- Fixed subject-verb agreement: "credential stacks are"
- Standardized sub-bullet indentation in step 1
- Split Cause section into two paragraphs for readability
- Upgraded NOTE to IMPORTANT and moved it above the reinstall bullet list in step 3 so readers see the Duo settings warning before proceeding
- Moved registry key block inside the IMPORTANT callout at the end of the article
- Confirmed Duo instructions external link resolves correctly
- Confirmed rendering in both light and dark mode
Local testing
- Dev server (
npm run start) — KB article rendered correctly; changes to Cause section, IMPORTANT callout placement, and registry block inside callout all confirmed visually - Pre-existing compilation errors in
docs/changetracker/8.1/integration/api/reference/are unrelated to this PR and did not affect the KB page
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adding new KB article to Password Reset that covers a scenario when Duo MFA, Password Policy Enforcer Client and Password Reset Client are all installed in the environment.