Skip to content

Conversation

@tobias-johansson
Copy link
Contributor

No description provided.

@tobias-johansson
Copy link
Contributor Author

Copy link
Contributor

@LinneaAndersson LinneaAndersson Nov 17, 2022

Choose a reason for hiding this comment

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

I would have moved this one to the first line, so that it says:

On <<composite-databases, composite databases>> the `id()` function should be used with caution. Instead, it is recommended to use <<functions-element-id, `elementId()`>>.

@neo-technology-commit-status-publisher
Copy link
Collaborator

Looks like you've updated the documentation!

Check out your changes at https://neo4j-docs-cypher-161.surge.sh

Copy link
Collaborator

@JPryce-Aklundh JPryce-Aklundh left a comment

Choose a reason for hiding this comment

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

LGTM! (Just a small suggestion)

@JPryce-Aklundh JPryce-Aklundh merged commit 760ddd3 into neo4j:dev Nov 24, 2022
@JPryce-Aklundh JPryce-Aklundh deleted the fabric-id-function/dev branch November 24, 2022 15:16
JPryce-Aklundh added a commit that referenced this pull request Nov 24, 2022
JPryce-Aklundh added a commit that referenced this pull request Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5.1 Neo4j version 5.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants