Skip to content
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

[#55950] updated documentation for storage health checks #15981

Conversation

Kharonus
Copy link
Member

#55950

WAT?

  • updated text and overall structure
  • missing current screenshots

Hint

  • @MayaBerd we need to talk about some headings, the future of this page and the screenshots, once you are back.

@Kharonus Kharonus requested a review from MayaBerd June 26, 2024 12:03
@Kharonus Kharonus self-assigned this Jun 26, 2024
@MayaBerd MayaBerd changed the base branch from dev to release/14.3 July 4, 2024 09:16
@Kharonus Kharonus force-pushed the implementation/55950-update-documentation-for-connection-tests branch from 939de9b to 840f0d7 Compare July 5, 2024 11:57
Kharonus and others added 4 commits July 5, 2024 14:42
- added more comprehensible explanation of what the health status is and
  how it is used for troubleshooting
- updated keywords and headlines
Copy link
Contributor

@MayaBerd MayaBerd left a comment

Choose a reason for hiding this comment

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

thanks @Kharonus, i've committed my changes

@Kharonus
Copy link
Member Author

I am satisfied with the new screenshots and changes by @MayaBerd , tusen takk!

what about you, @dominic-braeunlein ?

Copy link
Contributor

@dominic-braeunlein dominic-braeunlein left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Kharonus Kharonus merged commit 21c720e into release/14.3 Jul 11, 2024
7 checks passed
@Kharonus Kharonus deleted the implementation/55950-update-documentation-for-connection-tests branch July 11, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants