Skip to content

EPP_minor_updates_05052026#855

Merged
krzysztofstaszalek merged 2 commits into
devfrom
epp_minor05052026
May 5, 2026
Merged

EPP_minor_updates_05052026#855
krzysztofstaszalek merged 2 commits into
devfrom
epp_minor05052026

Conversation

@krzysztofstaszalek
Copy link
Copy Markdown
Collaborator

EPP

List of minor updates, limitations define for 2605 scope etc.

@krzysztofstaszalek krzysztofstaszalek requested a review from a team as a code owner May 5, 2026 11:05
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Documentation PR Review

Editorial Review

docs/endpointprotector/admin/cap_module/capai.md

  • Clarity — Lines 32–33: The prerequisite items "use latest EPP Server version" and "use latest EPP Client version" are phrased as imperative verb phrases, while the adjacent items ("CAP license with the Content Aware Protection (CAP) module enabled" and "DPI (Deep Packet Inspection) module enabled") use noun phrases. The inconsistency breaks the parallel structure of the prerequisites list. Suggested fix: "EPP Server (latest version)" and "EPP Client (latest version)."

  • Completeness — Lines 32–33: Replacing specific version numbers with "latest" removes the baseline admins need to verify their installation qualifies. Readers have no reference point for what "latest" means or where to check. Suggested fix: link to release notes or the community portal, or add a minimum version alongside a note that newer versions are also supported. For example: "EPP Server version 2509 or newer (latest recommended)."

docs/endpointprotector/admin/cap_module/capmodule.md

No issues found.

docs/endpointprotector/admin/cap_module/deeppacket.md

No issues found.

docs/endpointprotector/admin/cap_module/newoutlook.md

No issues found.

docs/endpointprotector/admin/dc_module/globalsettings.md

  • Completeness — Line ~653: The Per document sub-bullet introduces the acronym string "MIP/NDC" ("scans the entire file, including metadata (MIP/NDC)") without expanding either acronym on first use. Per Netwrix writing standards, all acronyms must be spelled out at first use. Neither "MIP" nor "NDC" is defined anywhere in this document. Suggested fix: expand inline, e.g., "including metadata such as MIP (Microsoft Information Protection) labels and NDC (Netwrix Data Classification) tags" — or whatever the correct full forms are.

  • Clarity — Line ~917: The debug mode warning says "when replicating an issue for further Netwrix Support Escalations." "Netwrix Support Escalations" is unexpectedly capitalized, making it read like a product feature name rather than a support process. The word "further" is also ambiguous — it's unclear whether it modifies "replicating" or "Escalations." Suggested fix: "when replicating an issue to report to Netwrix Support" or "when working with Netwrix Support to diagnose an issue."

docs/endpointprotector/admin/denylistsallowlists/allowlists.md

No issues found.

docs/endpointprotector/admin/denylistsallowlists/denylists.md

No issues found.

Summary

4 editorial suggestions across 2 files. 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

github-actions Bot commented May 5, 2026

⚠️ Broken Anchor Links

2 broken anchor link(s) found — these will cause the build to fail.

  docs/endpointprotector/admin/dc_module/globalsettings.md:285
    See the [Intercept VPN Traffic](#intercept-vpn-traffic) topic.
    #intercept-vpn-traffic not found in docs/endpointprotector/admin/dc_module/globalsettings.md
    Available: #global-settings · #client-settings · #client-mode · #dpi-conguration · #intercept-vpn-trac · #smart-dpi-log-throttling · #bypass-log-reporting-frequency · #timeout-period-for-bypassed-websites · #handling-of-bypassed-domains-and-applications · #memory-and-disk-persistence · #clearing-bypass-state · #using-wireshark-for-network-trac-analysis · #file-tracing-and-shadowing · #block-print-from-browsers · #conguring-gpo-for-browser-extensions · #ignore-virtual-printers · #congure-max-file-size · #outside-hours-and-outside-network · #transfer-limit · #debug-logging · #debug-logging-usage · #debug-logging-actions · #data-obfuscation-rules · #enforced-encryption-easylock-settings · #additional-information · #display-settings
  docs/endpointprotector/admin/dc_module/globalsettings.md:342
    See [Using Wireshark for Network Traffic Analysis](#using-wireshark-for-network-traffic-analysis).
    #using-wireshark-for-network-traffic-analysis not found in docs/endpointprotector/admin/dc_module/globalsettings.md
    Available: #global-settings · #client-settings · #client-mode · #dpi-conguration · #intercept-vpn-trac · #smart-dpi-log-throttling · #bypass-log-reporting-frequency · #timeout-period-for-bypassed-websites · #handling-of-bypassed-domains-and-applications · #memory-and-disk-persistence · #clearing-bypass-state · #using-wireshark-for-network-trac-analysis · #file-tracing-and-shadowing · #block-print-from-browsers · #conguring-gpo-for-browser-extensions · #ignore-virtual-printers · #congure-max-file-size · #outside-hours-and-outside-network · #transfer-limit · #debug-logging · #debug-logging-usage · #debug-logging-actions · #data-obfuscation-rules · #enforced-encryption-easylock-settings · #additional-information · #display-settings

Auto-Fix Summary

153 issues fixed, 12 skipped across 7 files

Category Fixes
Contractions 50
Removed filler 11
Substitutions 15
AllowsYouTo (rewrite) 8
BoilerplateCrossRef (rewrite) 3
Contractions (rewrite) 1
DesiredAsAdjective (rewrite) 2
FirstPersonPlural (rewrite) 6
FollowTheStepsTo (rewrite) 1
FormalHedging (rewrite) 1
Idioms (rewrite) 1
ImpersonalFiller (rewrite) 1
NoteThat (rewrite) 1
OnceUsage (rewrite) 6
OxfordComma (rewrite) 13
PlainTextAdmonition (rewrite) 1
QuestionHeadings (rewrite) 1
ReferToTheFollowing (rewrite) 1
TypeVsEnter (rewrite) 2
WeakLinkText (rewrite) 5
Dale: minimizing-difficulty 1
Dale: passive-voice 14
Dale: positional-references 6
Dale: wordiness 2
Skipped (needs manual review) Reason
docs/endpointprotector/admin/cap_module/capai.md:41 — Netwrix.FirstPerson The 'I.' is a Roman numeral section label ('I. Simple CAP Policy...'), not the first-person pronoun. False positive.
docs/endpointprotector/admin/cap_module/capmodule.md:40 — Netwrix.OxfordComma 'applications, reported or blocked file names display...' — the comma after 'applications' separates a dependent clause ('when using... sync applications') from the main clause ('reported or blocked file names display...'). 'reported or blocked' is a two-item compound modifier, not a three-item list. False positive.
docs/endpointprotector/admin/dc_module/globalsettings.md:275 — Netwrix.OxfordComma 'if enabled, network and browser traffic can be inspected' — the comma separates the conditional clause 'if enabled' from the main clause. 'network and browser' is a two-item compound, not a three-item list. False positive.
docs/endpointprotector/admin/dc_module/globalsettings.md:421 — Netwrix.OxfordComma Image alt text 'If enabled, network and browser traffic can be inspected for content' — same false positive as line 275; comma separates conditional clause.
docs/endpointprotector/admin/denylistsallowlists/allowlists.md:234 — Netwrix.FirstPersonPlural Three violations flagged for 'us' at line 234, which is a table row containing domain names: 'box.com.us', 'Sub.box.com.us', 'box1.com.us'. The 'us' is a country-code TLD in domain names, not first-person plural. False positives.
docs/endpointprotector/admin/denylistsallowlists/allowlists.md:234 — Netwrix.FirstPersonPlural Duplicate of above — 'us' in 'Sub.box.com.us' is a domain TLD, not first-person plural. False positive.
docs/endpointprotector/admin/denylistsallowlists/allowlists.md:234 — Netwrix.FirstPersonPlural Duplicate of above — 'us' in 'box1.com.us' is a domain TLD, not first-person plural. False positive.
docs/endpointprotector/admin/cap_module/capai.md:9 — Dale: passive-voice Modal passive 'can be fully managed' is standard product-capability phrasing; rewrite would change emphasis
docs/endpointprotector/admin/cap_module/newoutlook.md:11 — Dale: passive-voice 'can't be restricted' — ambiguous actor (Microsoft platform constraint, not user action); skip to avoid altering meaning
docs/endpointprotector/admin/cap_module/newoutlook.md:177 — Dale: passive-voice 'must be repeated' describes a characteristic of the method rather than a user action; rewrite would be less natural
docs/endpointprotector/admin/dc_module/globalsettings.md:289 — Dale: minimizing-difficulty 'seamless integration' describes a product characteristic, not a user task difficulty; rule targets user task minimization
docs/endpointprotector/admin/denylistsallowlists/denylists.md:386 — Dale: passive-voice 'This enable granular control' is a grammatical error (not passive voice); outside Dale rule scope

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

@jth-nw jth-nw temporarily deployed to development May 5, 2026 11:25 — with GitHub Actions Inactive
@krzysztofstaszalek krzysztofstaszalek merged commit 99ab69a into dev May 5, 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