Skip to content

Conversation

@stefano-ottolenghi
Copy link
Contributor

This all started with me using this feature and noticing that the documented examples triggered a warning.

Screenshot from 2025-04-19 08-57-24

Digging further into the feature and thus the docs, I took the chance to polish examples (syntax highlighting, avoid scrolling) and a few wordings.

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.

I added some comments, but as I said, I would ask the Clustering team to review this PR.

@stefano-ottolenghi stefano-ottolenghi force-pushed the improve-restore-backup branch from bb0baca to a7e102d Compare May 7, 2025 06:40
@stefano-ottolenghi
Copy link
Contributor Author

Changing base branch on this PR is a mess, so #2319 supersedes this PR onto the right cypher-25 branch.
This PR now only fixes the slashes bug.

@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented May 7, 2025

Thanks for the documentation updates.

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

@stefano-ottolenghi stefano-ottolenghi merged commit 407a771 into dev May 7, 2025
4 checks passed
@stefano-ottolenghi stefano-ottolenghi deleted the improve-restore-backup branch May 7, 2025 06:42
stefano-ottolenghi added a commit that referenced this pull request May 7, 2025
This all started with me using this feature and noticing that the
documented examples triggered a warning.

![Screenshot from 2025-04-19
08-57-24](https://github.com/user-attachments/assets/a18a0724-9a65-4f65-957c-7791eb82e9f8)

Digging further into the feature and thus the docs, I took the chance to
polish examples (syntax highlighting, avoid scrolling) and a few
wordings.
stefano-ottolenghi added a commit that referenced this pull request May 7, 2025
Spinoff of #2287 , on the `cypher-25` base branch as the option is deprecated then.
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.

5 participants