Feature/ppe april2 jm2#830
Conversation
- Add new Single Sign On category and article for PolicyPak Cloud KB - Add 6 SSO screenshots to static/images - Update bug reporting, component versioning, and OS support sections in ppe-supportability.md across versions 10.2, 11.0, and 11.1 Generated with AI Co-Authored-By: Claude Code <ai@netwrix.com>
Apply the three content updates (bug reporting policy, component versioning paragraph, OS support section) to the canonical file at docs/kb/passwordpolicyenforcer/ppe-supportability.md. Remove the force-tracked versioned copies — these are gitignored by design and will be regenerated by the KB copy script at build time. Generated with AI Co-Authored-By: Claude Code <ai@netwrix.com>
jeremymoskowitz-netwrix
had a problem deploying
to
development
GitHub Actions
Error
Auto-Fix Summary9 issues fixed, 3 skipped across 1 files
Ask |
|
@claude fix all issues then publish. |
- Promote bold-text section headers to proper ## headings - Remove duplicate 3-business-day SLA note from Step 1 body - Fix third-person "user" → second-person "you" in Step 4 - Fix mixed second/third person in Step 7 intro sentence - Clarify activation link expiry: "48 hours after it is sent" - Fix "continue" → "sign in to PolicyPak Cloud" in Step 7 - Add 1Secure context sentence at top of Step 2 - Add OIDC provider note in Step 5 - Rewrite "Use the following guides" → "Before continuing, register..." in Step 5 Co-Authored-By: Claude <noreply@anthropic.com>
Auto-Fix Summary8 issues fixed, 1 skipped across 1 files
Ask |
hilram7
left a comment
hilram7
left a comment
There was a problem hiding this comment.
Ran Vale, Dale, and Derek on ppe-supportability.md. The sso.md file was reviewed separately — see notes below. I'll push a follow-up commit with all fixes.
ppe-supportability.md
Vale:
- Lines 123, 140 — first person plural (we). Rewrite without "we", "our", "us", or "ours."
Dale:
- Line 104 — passive voice: "Running mismatched component versions is not a supported configuration." Rewrite in active voice.
- Line 140 — passive voice: "PPE's components...are developed and tested as a single package." Rewrite in active voice.
- Line 140 — wordiness: "You will have the best experience using all the components from a single version." Suggest trimming.
Derek:
- Line 26 — title contains a product name. Product names belong in the products frontmatter field. Suggest "Understanding PPE Supportability" or "Password Policy Enforcer
Support Levels."
I'll push a follow-up commit with all fixes.
sso.md
This file lives under docs/policypak/ and follows that product's own doc structure — standard KB rules don't fully apply. The one Vale warning (line 44, first person my) is a
false positive — it flags the button label text "Activate my Netwrix account," not author-written first person. No fixes needed.
Note for future PRs: PPE and PolicyPak are separate products — keeping changes scoped to one product per PR makes reviews easier to track.
I'll push a follow-up commit with all fixes.
Ran Vale, Dale, and Derek. Applied all fixes. - Removed product name from title: "Understanding PPE Supportability" - Rewrote passive voice on lines 104 and 140 - Replaced first person plural (we/our) on lines 123 and 140 - Trimmed wordiness on line 140 Local dev and build tests passed with no errors.
Documentation PR ReviewEditorial Reviewdocs/policypak/deliverymethods/cloud/knowledgebase/singlesignon/sso.md
Summary3 editorial suggestions across 1 file. Vale and Dale issues are auto-fixed separately. What to do next: Comment
You can ask Claude anything about the review or about Netwrix writing standards.
|
Auto-Fix Summary2 issues fixed, 1 skipped across 1 files
Ask |
4 participants
Added one PP article about SSO (ships May 4th, but approve asap.) And some updates to PPE supportability Policies.