Skip to content

Deprecate raw initial_bookmarks for bookmark_manager - #1196

Merged
robsdedude merged 3 commits into
neo4j:6.xfrom
robsdedude:deprecate-bmm-raw-initial-bookmarks
May 23, 2025
Merged

Deprecate raw initial_bookmarks for bookmark_manager#1196
robsdedude merged 3 commits into
neo4j:6.xfrom
robsdedude:deprecate-bmm-raw-initial-bookmarks

Conversation

@robsdedude

@robsdedude robsdedude commented May 22, 2025

Copy link
Copy Markdown
Member

Deprecate passing raw sting bookmarks as initial_bookmarks to GraphDatabase.bookmark_manager().
Use a neo4j.Bookmarks object instead.

Amends #1175

@robsdedude
robsdedude force-pushed the deprecate-bmm-raw-initial-bookmarks branch from c4dd0f1 to a858acb Compare May 22, 2025 10:11
Deprecate passing raw sting bookmarks as `initial_bookmarks` to
`GraphDatabase.bookmark_manager()`.
Use a `neo4j.Bookmarks` object instead.
@robsdedude
robsdedude force-pushed the deprecate-bmm-raw-initial-bookmarks branch from a858acb to 0f2f787 Compare May 22, 2025 11:50

@AndyHeap-NeoTech AndyHeap-NeoTech left a comment

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.

Put it in a box and ship it.

@robsdedude
robsdedude enabled auto-merge (squash) May 23, 2025 10:49
@robsdedude
robsdedude merged commit 381616e into neo4j:6.x May 23, 2025
@robsdedude
robsdedude deleted the deprecate-bmm-raw-initial-bookmarks branch May 23, 2025 11:45
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.

2 participants