Skip to content

Conversation

@rsill-neo4j
Copy link
Contributor

No description provided.

Copy link
Contributor

@lidiazuin lidiazuin left a comment

Choose a reason for hiding this comment

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

Thanks a lot! Left some comments

[[ogm-installation]]
:description: This page describes how to install the OGM in Neo4j GraphQL and includes examples of how to use it.
= Installation
:page-aliases: ogm/examples/index.adoc, omg/examples/rest-api.adoc, omg/examples/custom-resolvers.adoc
Copy link
Contributor

Choose a reason for hiding this comment

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

https://neo4j.com/docs/graphql/3/ogm/examples/custom-resolvers/ is already listed in the three versions and it's taking to https://neo4j.com/docs/graphql/5/custom-resolvers/ . We need to remove it from here or it will cause a loop.

omg/examples/rest-api.adoc doesn't have a substitute, so I wouldn't redirect to /ogm/installation.adoc

All in all, I think we could go only with ogm/examples/index.adoc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

true and true, updated

rsill-neo4j and others added 2 commits March 19, 2024 08:17
Co-authored-by: Lidia Zuin <102308961+lidiazuin@users.noreply.github.com>
Copy link
Contributor

@lidiazuin lidiazuin left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!!

@rsill-neo4j rsill-neo4j merged commit 34915a0 into 5.x Mar 19, 2024
@rsill-neo4j rsill-neo4j deleted the add-missing-aliases branch April 2, 2025 09:00
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.

4 participants