-
Notifications
You must be signed in to change notification settings - Fork 202
[ADR-001] bookmark manager #771
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 31 commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
b2b6f42
Initial draft of bookmark manager
robsdedude 1ecd68f
Fix unit tests for config
robsdedude 32273f9
TestKit backend support for bookmark manager
robsdedude 2279b67
Fix bookmark manager
robsdedude e696f07
Add support for `ignore_bookmark_manager` session config option
robsdedude 693b9b5
Lock bookmark manager per db
robsdedude 4ec20b3
Add `forget` method to bookmark manager
robsdedude ad092c9
Add API docs for the bookmark manager
robsdedude e804076
Implement bookmark_consumer and remove must_included_databases
robsdedude bb0efae
Make Bookmarks not iterable
robsdedude 206bfbe
Merge branch '5.0' into bookmark-manager
robsdedude ecd70db
TestKit backend support for BMM extension functions
robsdedude 90d6abc
Fix corner-case usages of the BMM within Session
robsdedude bb07e19
Unit tests for BMM
robsdedude 4712348
Fix TestKit backend handling of BMM config options
robsdedude 8902770
Rename bookmark manager hooks
robsdedude 87f4eaf
TestKit add Optimization:MinimalBookmarksSet flag
robsdedude 1e0d440
Add performance warning for when enabling the BMM
robsdedude bd5e6a9
Merge branch '5.0' into bookmark-manager
robsdedude 97189f0
It's manager... not manger, duh!
fbiville 44eef98
Revert unrelated change to tests
robsdedude c166e2b
Clean-up
fbiville 3891c4e
Clean-up
bigmontz 517c07d
Fix session.last_bookmarks() leaking bookmarks from BMM
robsdedude d72b747
Merge branch '5.0' into bookmark-manager
robsdedude f9070cf
Move BMM config to session level + mark experimental
robsdedude bfbf7e5
TestKit: support session-level BMM config
robsdedude 9f2a476
Code-style: expand abbreviations
robsdedude af13113
Fix spelling
robsdedude fe7a7bd
Merge branch '5.0' into bookmark-manager
robsdedude 17bdfda
Merge branch '5.0' into bookmark-manager
robsdedude e7dfb59
Merge branch '5.0' into bookmark-manager
robsdedude File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.