Skip to content

Conversation

@angrykoala
Copy link
Member

No description provided.

@angrykoala angrykoala requested a review from rsill-neo4j March 21, 2024 13:57
@angrykoala angrykoala linked an issue Mar 21, 2024 that may be closed by this pull request
[source, graphql, indent=0]
----
query {
name(value: "aca")
Copy link
Contributor

Choose a reason for hiding this comment

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

what's "aca"?

Copy link
Member Author

@angrykoala angrykoala Mar 26, 2024

Choose a reason for hiding this comment

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

It's a random text. I will change for something better

angrykoala and others added 2 commits March 26, 2024 11:02
Co-authored-by: Richard Sill <156673635+rsill-neo4j@users.noreply.github.com>
@angrykoala angrykoala requested a review from rsill-neo4j March 26, 2024 10:04
}
----

The following GraphQL query returns the parameter `value`:
Copy link
Contributor

Choose a reason for hiding this comment

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

is the parameter name or value? i think i'm confused

Copy link
Member Author

Choose a reason for hiding this comment

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

Here I'm trying to reference the parameter (value) passed to the query (name).


[source, graphql, indent=0]
----
query {
Copy link
Contributor

Choose a reason for hiding this comment

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

another point of confusion on my part: how does this query relate to the query above with the cypher directive in it?

Copy link
Member Author

Choose a reason for hiding this comment

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

In this query we are passing a parameter called value that is caught by the @cypher directive (see the $value)

Co-authored-by: Richard Sill <156673635+rsill-neo4j@users.noreply.github.com>
@neo-technology-commit-status-publisher
Copy link
Collaborator

neo-technology-commit-status-publisher commented Apr 8, 2024

Thanks for the documentation updates.

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

@rsill-neo4j rsill-neo4j merged commit d13cd5c into 5.x Apr 8, 2024
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.

Cypher Directive documentation improvements

4 participants