Add Change Tracker KB batch 4b - #1268
Merged
Merged
Conversation
…lete Migrates 5 KB articles from the staging repo for Denis Goskolli's SME review, with content depth work already applied via kb-writer. Two Batch 4b candidates (File Content Max, File Hash Max for Extension) are held pending clarification on the current agent config file format, and are not included here. Vale/Dale/Derek lint pass (kb-pr-open) not yet run — picking back up in a later session.
…b articles Applies the kb-pr-open Vale/Dale/Derek findings: bolds Command Prompt where opened, fixes passive voice and positional references, corrects a code-fence language mismatch, and reorders the bulk-email Overview to lead with the goal sentence. Standardizes all SME Review Needed inline flags to a consistent bracketed format. Converts the MongoDB timeout article's backup instruction to an IMPORTANT callout rather than its own step.
Adjusted paths/key names to match current version of Change Tracker
Added the key found in the standard Change Tracker installation. Unsure what causes the duplicate key issue described in this article
dgoskolli
reviewed
Jul 28, 2026
dgoskolli
reviewed
Jul 28, 2026
dgoskolli
left a comment
dgoskolli
left a comment
Contributor
There was a problem hiding this comment.
Review completed and articles have been updated with the correct details/comments
… review fixes for batch 4b Turns SME (Denis Goskolli) review comments into actual article content where his feedback was left as a comment but never committed: confirms Account Lockout overrides the now-unused IP-blocking behavior in stop-blocking-ip-address.md, and clears resolved SME placeholders in mongodb-connection-timeout-socketexception-10060.md and turn-off-bulk-email-notifications.md (plus best-judgment content for the still-open batching-rationale question). Fixes a broken numbered list in http-500-19-x-frame-options-duplicate-entry.md caused by an unindented code block, and adds a best-judgment Cause explanation where the SME was unsure of the root cause. Renames two article titles from generic "Rolling-Log Fix:" to the established "<Component> Log Fix:" sibling convention. Applies remaining Dale (passive voice, wordiness, misplaced modifiers) and Derek (title format, product-name consistency, formatting) fixes across all five changed KB articles.
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
2 participants
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
Adds 5 new Change Tracker KB articles migrated from the staging repo for Denis Goskolli's SME review.
Changes
Testing
SME Review Needed
stop-blocking-ip-address.md— The article states
lockoutenabledlocks the account after repeated failed logins but doesn't confirm the relationship to Change Tracker's default IP-blocking behavior. Please confirm:http-500-19-x-frame-options-duplicate-entry.md— The Cause doesn't explain why the upgrade produces a duplicate
X-Frame-Optionsentry instead of replacing it, and no currentX-Frame-Optionsvalue is available to include in the Resolution. Please confirm:mongodb-connection-timeout-socketexception-10060.md— The Cause explanation (connection pool exhaustion) reflects standard MongoDB .NET driver behavior but should be verified against Change Tracker's actual implementation. Please confirm:
turn-off-bulk-email-notifications.md— Two open items:
(1) Config: Confirm the current file and key name for disabling bulk email notifications (
localhost.json/enableBulkNotificationsunverified against 8.2).(2) Content: What determines when changes are grouped into one bulk email (time window, report run, device, etc.), why a customer would want this off, and whether disabling it meaningfully increases email volume? Please confirm:
Closes #1269