Skip to content

Updated Customer Portal page and images#906

Merged
hilram7 merged 13 commits into
devfrom
training-may13a
May 14, 2026
Merged

Updated Customer Portal page and images#906
hilram7 merged 13 commits into
devfrom
training-may13a

Conversation

@robin-stefani
Copy link
Copy Markdown
Collaborator

Updated Customer Portal page and images

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

7 issues fixed, 3 skipped across 1 files

Category Fixes
Contractions 1
FirstPersonPlural (rewrite) 1
FollowTheStepsTo (rewrite) 1
OxfordComma (rewrite) 1
Dale: passive-voice 2
Dale: wordiness 1
Skipped (needs manual review) Reason
docs/customer/portal/index.md:30 — Netwrix.FirstPerson The phrase 'I'm not a robot' is the literal label of the Google reCAPTCHA checkbox UI element. Rewriting it would misrepresent the actual interface text the user sees.
docs/customer/portal/index.md:26 — Dale: passive-voice 'known to ' is stative and difficult to rewrite cleanly without altering meaning
docs/customer/portal/index.md:58 — Dale: passive-voice 'is known to ' is stative; rewriting risks changing the intended check

Ask @claude on this PR if you'd like an explanation of any fix.

Clarified password requirements and updated sign-in instructions.
@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

3 issues fixed, 1 skipped across 1 files

Category Fixes
Dale: passive-voice 2
Dale: wordiness 1
Skipped (needs manual review) Reason
docs/customer/portal/index.md:30 — Netwrix.FirstPerson The 'I'm' is part of the literal UI label 'I'm not a robot' on the Google reCAPTCHA checkbox. Rewriting would misrepresent the verbatim UI text the user must locate and click.

Ask @claude on this PR if you'd like an explanation of any fix.

Revised password requirements for clarity and improved instructions for account verification.
@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 1 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:34 — Netwrix.FirstPerson The 'I'm' is part of the verbatim UI label on the reCAPTCHA checkbox (I'm not a robot). Rewriting would either alter literal UI text the reader must recognize on screen, or rephrase around it while leaving the bolded UI text — and the Vale violation — in place.

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

1 issues fixed, 4 skipped across 1 files

Category Fixes
Dale: wordiness 1
Skipped (needs manual review) Reason
docs/customer/portal/index.md:34 — Netwrix.FirstPerson The 'I'm' is part of the verbatim UI label 'I'm not a robot' — the actual reCAPTCHA checkbox text. Rewording it would misrepresent the UI element the user must click.
docs/customer/portal/index.md:26 — Dale: idioms 'on file' is a standard business term, not a culturally specific idiom; rewriting could obscure meaning
docs/customer/portal/index.md:40 — Dale: wordiness 'you will receive an email' uses future tense for a genuinely future event; rewriting risks changing meaning
docs/customer/portal/index.md:60 — Dale: wordiness 'any trouble' is idiomatic English and removing 'any' offers negligible benefit

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 1 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:34 — Netwrix.FirstPerson The phrase 'I'm not a robot' is the literal UI label of the reCAPTCHA checkbox. Rewriting it would make the documentation inaccurate, since the user must identify and check the box with that exact text.

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 1 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:34 — Netwrix.FirstPerson The flagged 'I'm' is part of the literal UI label 'I'm not a robot' (the reCAPTCHA checkbox text). UI element labels must be quoted verbatim; rewriting would misrepresent the on-screen text the reader needs to click.

Ask @claude on this PR if you'd like an explanation of any fix.

Updated password requirements and rephrased account verification instructions for clarity.
@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 1 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:34 — Netwrix.FirstPerson The 'I'm' is part of the literal Google reCAPTCHA UI element label ('I'm not a robot') that users must click. Rewriting would misrepresent the actual UI label and reduce instructional accuracy.

Ask @claude on this PR if you'd like an explanation of any fix.

Revised password instructions for clarity and added 'Confirm Password' field.
@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 4 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:35 — Netwrix.FirstPerson The flagged 'I'm' is part of the literal reCAPTCHA UI label 'I'm not a robot' that users must locate on the registration page. Rewording would make the instruction inaccurate.
docs/customer/portal/index.md:26 — Dale: idioms 'on file' is standard business English, not a colorful idiom; replacement could subtly change meaning
docs/customer/portal/index.md:64 — Dale: idioms 'on file' is standard business English, not a colorful idiom; replacement could subtly change meaning
docs/customer/portal/index.md:63 — Dale: wordiness Sentence is slightly verbose but rewriting risks losing clarity for newer users per audience guidance in docs/CLAUDE.md

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/customer/portal/index.md

  • Clarity — Line 34: "Renter the password" appears to be a typo for "Re-enter". Suggested fix: "Confirm Password – Re-enter the password."
  • Consistency — Line 55: "Ready to use page" reads as the name of a UI page, but it isn't bolded the way other UI elements in this procedure are (for example, Sign In and Verify Your Account). Readers scanning the step may not realize it's a page label. Suggested fix: bold and use the exact capitalization of the page title, for example, "redirects you to the Ready to Use page."
  • Clarity — Line 57: "From the Customer Portal, you can return to the main website by clicking the logo in the upper-left corner. Then you can access the Customer Portal through the user icon." The word "Then" implies these two actions happen in sequence, but they are two independent navigation paths. Suggested fix: "From the Customer Portal, you can return to the main website by clicking the logo in the upper-left corner. To return to the Customer Portal from the website, click the user icon."
  • Consistency — Line 63: "clicking the Verify Your Account link" — the UI element is bolded in step 8 (Verify Your Account) but not here. Suggested fix: "clicking the Verify Your Account link" to match the formatting used in the procedure.
  • Completeness — Line 28: The new Password requirements list adds rules the reader must follow but doesn't note whether the portal displays these requirements on the Sign Up form or whether the user must memorize them. If the form shows the rules, a brief mention helps the reader trust what they see; if not, calling that out prevents surprise. Suggested fix: add a short lead-in such as "Password – Enter a password that meets the following requirements (also displayed on the Sign Up form):".

Summary

5 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
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 3 skipped across 1 files

Skipped (needs manual review) Reason
docs/customer/portal/index.md:35 — Netwrix.FirstPerson The flagged 'I'm' is part of the literal UI label 'I'm not a robot' (Google reCAPTCHA checkbox text). Rewriting would misrepresent the actual UI element the user must interact with.
docs/customer/portal/index.md:64 — Dale: misplaced-modifiers 'on file' at end of sentence is awkwardly placed but reordering risks changing meaning (e.g., 'has on file the corporate email address you used to register' is stilted)
docs/customer/portal/index.md:63 — Dale: wordiness Long sentence but each clause adds useful specificity; tightening risks dropping context the reader needs

Ask @claude on this PR if you'd like an explanation of any fix.

@robin-stefani robin-stefani marked this pull request as ready for review May 13, 2026 20:21
@robin-stefani robin-stefani requested a review from a team as a code owner May 13, 2026 20:21
Copy link
Copy Markdown
Collaborator

@hilram7 hilram7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Notes

  • Proofread docs/customer/portal/index.md — no further revision needed.
  • All screenshots updated and links redirect correctly, confirmed via local dev build.

@hilram7 hilram7 merged commit 0a2d133 into dev May 14, 2026
10 checks passed
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.

3 participants