-
Notifications
You must be signed in to change notification settings - Fork 14
New KB on account exclusions #82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…is no domain trust relationship
This comment was marked as outdated.
This comment was marked as outdated.
Corrected warning admonition and related links header
This comment was marked as outdated.
This comment was marked as outdated.
Took some suggestions from Claude and made minor improvements
|
Pull Request Review: Account Exclusions KB Article SUMMARY: STRENGTHS:
DOCUMENTATION QUALITY ISSUES: PowerShell Code Block at lines 42-49:
Numbered List Formatting at lines 54-56:
Configuration File Path:
Frontmatter:
SECURITY CONSIDERATIONS: TECHNICAL ACCURACY: RECOMMENDATIONS SUMMARY: High Priority:
Medium Priority: Low Priority: APPROVAL STATUS: |
typo in powershell command
This comment was marked as resolved.
This comment was marked as resolved.
docs/kb/activitymonitor/monitoring-platforms-and-storage/multi-domain-account-exclusions.md
Outdated
Show resolved
Hide resolved
docs/kb/activitymonitor/monitoring-platforms-and-storage/multi-domain-account-exclusions.md
Outdated
Show resolved
Hide resolved
|
The document looks good! You can ignore the doc reviewer stuff - I made some comments suggesting some changes (I want the doc reviewer to do it this way in the future) |
Revised symptom phrasing
This comment was marked as resolved.
This comment was marked as resolved.
Proofread — improved formatting. minor word choice revision
This comment was marked as resolved.
This comment was marked as resolved.
docs/kb/activitymonitor/monitoring-platforms-and-storage/multi-domain-account-exclusions.md
Outdated
Show resolved
Hide resolved
docs/kb/activitymonitor/monitoring-platforms-and-storage/multi-domain-account-exclusions.md
Outdated
Show resolved
Hide resolved
Fix YAML front matter parsing error — indented the block value of >-
This comment was marked as resolved.
This comment was marked as resolved.
Fixed typo, updated title, and standardized formatting
New KB on account exclusions for multidomain environments when there is no domain trust relationship