Skip to content

docs(1secure): clarify when New status changes to Healthy - #1699

Merged
james-s-anderson merged 1 commit into
devfrom
1secure-statuses-note
Oct 19, 2026
Merged

james-s-anderson merged 1 commit into
devfrom
1secure-statuses-note

Conversation

@james-s-anderson

Copy link
Copy Markdown
Contributor

Updates the note under the diagram on the Statuses page: the New status changes to Healthy when the first batch of data arrives from the environment without error. Removes the sentence about how long the change can take.

Generated with AI

Co-Authored-By: Claude Code ai@netwrix.com

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@james-s-anderson
james-s-anderson requested a review from a team as a code owner October 14, 2026 11:36
@github-actions

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/1secure/admin/statuses.md

  • Completeness — Lines 16–17: The earlier note told readers how long the change from New to Healthy can take (from about a minute to several hours, depending on environment size), and this edit removes that. Without it, readers can't tell whether a source that has shown New for an hour is still working normally or has a problem. If the time frame still holds, keep it. Suggested fix: "The New status changes to Healthy when the first batch of data arrives from the environment without error. This can take from about a minute to several hours, depending on the size of the environment."
  • Completeness — Line 17: "without error" suggests the status can change to something other than Healthy, but the note doesn't say what happens if the first batch of data has errors. The statuses are listed only in an image, so readers can't easily look up the other status as text. Suggested fix: add a sentence that names the resulting status, for example: "If the data contains errors, the status changes to <status-name>." Replace <status-name> with the status the product actually shows.

Summary

2 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, 0 skipped across 1 files

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

@james-s-anderson
james-s-anderson merged commit f2162c0 into dev Oct 19, 2026
14 checks passed

This branch was successfully deployed

1 active deployment
development — 540d6041 Deployed Oct 14, 2026 by james-s-anderson via build-and-deploy #4531
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.

2 participants