Skip to content

Conversation

lidiazuin
Copy link
Contributor

No description provided.

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Thanks, @lidiazuin. I added some suggestions.

Copy link
Collaborator

Choose a reason for hiding this comment

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

One arrow is missing Bob (admin) -shares user credentials-> Alice (admin).
And also, maybe change query to queries for consistency

@lidiazuin lidiazuin requested a review from renetapopova August 12, 2025 12:08
Copy link
Collaborator

@NataliaIvakina NataliaIvakina left a comment

Choose a reason for hiding this comment

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

I like them!

@lidiazuin
Copy link
Contributor Author

Can I merge this PR or you have anything else to add, @renetapopova ?

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Looks great. Just one comment.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I am just wondering if having "Backup chain" on top will be perceived as the diagram's title.

@lidiazuin lidiazuin requested a review from renetapopova August 14, 2025 10:08
@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented Aug 14, 2025

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you, @lidiazuin.

@lidiazuin lidiazuin merged commit 3e14389 into neo4j:dev Aug 15, 2025
8 checks passed
@lidiazuin lidiazuin deleted the imgrefresh-batch1 branch August 15, 2025 08:33
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Aug 15, 2025
lidiazuin added a commit that referenced this pull request Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants