Skip to content

Prototype for moving List procedures page to Ops Manual#3003

Open
JPryce-Aklundh wants to merge 13 commits into
neo4j:devfrom
JPryce-Aklundh:procedure_playground_2
Open

Prototype for moving List procedures page to Ops Manual#3003
JPryce-Aklundh wants to merge 13 commits into
neo4j:devfrom
JPryce-Aklundh:procedure_playground_2

Conversation

@JPryce-Aklundh
Copy link
Copy Markdown
Contributor

@JPryce-Aklundh JPryce-Aklundh commented Apr 23, 2026

See corresponding Cypher Manual PR:

neo4j/docs-cypher#1531

@renetapopova
Copy link
Copy Markdown
Collaborator

Hey @JPryce-Aklundh, we think that we also need to move the CALL procedure page to the Operations manual, so we keep everything in one place.

Comment thread modules/ROOT/pages/procedures/index.adoc Outdated
@renetapopova renetapopova force-pushed the procedure_playground_2 branch from 1f75eda to 3ebee4f Compare May 14, 2026 08:59
@renetapopova renetapopova marked this pull request as ready for review May 14, 2026 10:48
@renetapopova renetapopova requested a review from Hunterness May 14, 2026 10:48
Copy link
Copy Markdown
Contributor

@Hunterness Hunterness left a comment

Choose a reason for hiding this comment

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

Review

Comment thread modules/ROOT/pages/procedures/show-procedures.adoc Outdated
Comment thread modules/ROOT/pages/procedures/show-procedures.adoc Outdated
Comment thread modules/ROOT/pages/procedures/show-procedures.adoc Outdated
Comment thread modules/ROOT/pages/procedures/show-procedures.adoc
Comment thread modules/ROOT/pages/procedures/show-procedures.adoc
Comment thread modules/ROOT/pages/procedures/show-procedures.adoc Outdated
@@ -0,0 +1,282 @@
:description: Information about how to use Neo4j procedures using Cypher's `CALL` clause.
= Call procedures
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

While I could look at this page and compare it with the current page, I think this is more of a Cypher Surface page so maybe they want to look at it?

Comment thread modules/ROOT/pages/procedures/call-procedures.adoc
Comment thread modules/ROOT/pages/procedures/call-procedures.adoc
Comment thread modules/ROOT/pages/procedures/call-procedures.adoc
@renetapopova renetapopova force-pushed the procedure_playground_2 branch from fcb8d97 to f878e6c Compare May 15, 2026 13:30
renetapopova and others added 2 commits May 15, 2026 14:44
Co-authored-by: Therese Magnusson <scout.therese@gmail.com>
Co-authored-by: Therese Magnusson <scout.therese@gmail.com>
@JoelBergstrand
Copy link
Copy Markdown
Contributor

@renetapopova is the move of the Call a procedure page a duplication of the information on the Cypher docs or is it intended to remove the page on the Cypher side?

I think it makes sense that the information is close to where the actual procedures are listed but if it is planned to be removed on the Cypher side I'd like to see some minimized version next to the other clauses.

Has the page been heavily edited from what was in the Cypher docs? At firsts glance it looks similar.

@renetapopova
Copy link
Copy Markdown
Collaborator

@renetapopova is the move of the Call a procedure page a duplication of the information on the Cypher docs or is it intended to remove the page on the Cypher side?

I think it makes sense that the information is close to where the actual procedures are listed but if it is planned to be removed on the Cypher side I'd like to see some minimized version next to the other clauses.

Has the page been heavily edited from what was in the Cypher docs? At firsts glance it looks similar.

The page in the Cypher manual will be removed. It hasn't been changed, just restyled a bit.

Copy link
Copy Markdown
Contributor

Thanks - I'll review it from that angle then 🙂

@renetapopova
Copy link
Copy Markdown
Collaborator

The test is failing because we need to update the path to the procedures in the test.

@JoelBergstrand
Copy link
Copy Markdown
Contributor

@renetapopova the preview pages are not loading nicely - is that my browsers fault?

Copy link
Copy Markdown
Contributor

@JoelBergstrand JoelBergstrand left a comment

Choose a reason for hiding this comment

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

I've only looked a call-procedures.adoc, looks good to me. I'd like a stump page Cypher clause side if @JPryce-Aklundh agrees otherwise we can do without.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2026.05 team-cypher-surface The PR should be reviewed by Team Cypher surface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants