Skip to content
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

feat(indexeddb): advanced cursor filtering impl #2066

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

borngraced
Copy link
Member

@borngraced borngraced commented Feb 19, 2024

This pull request adds several new filtering features to the mm2 indexedbdb crate, including limit, offset, with a minor fix for where_ condition/option.

closes: #2052, #2019

@borngraced borngraced added the in progress Changes will be made from the author label Feb 19, 2024
@borngraced borngraced self-assigned this Feb 19, 2024
@borngraced borngraced changed the title feat(indexeddb): advanced indexeddb cursor filtering impl feat(indexeddb): advanced cursor filtering impl Feb 19, 2024
Comment on lines 274 to 278
/// Continues the cursor according to the provided `CursorAction`. If the action
/// is `CursorAction::Continue`, the cursor advances to the next item. If the action is
/// `CursorAction::ContinueWithValue`, the cursor advances to the specified value. If the
/// action is `CursorAction::Stop`, the cursor is stopped, and subsequent calls to `next`
/// will return `None`.
Copy link
Member

Choose a reason for hiding this comment

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

Could you reformat this explanation into multiple lines (per line for each action)?

Copy link
Member

@onur-ozkan onur-ozkan left a comment

Choose a reason for hiding this comment

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

I will do the next review when this PR drops in-progress label and resolves the conflicts.

@borngraced borngraced added under review and removed in progress Changes will be made from the author labels Feb 23, 2024
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

Thank you for the PR! Only some minor comments and a question!

mm2src/mm2_db/src/indexed_db/drivers/cursor/cursor.rs Outdated Show resolved Hide resolved
mm2src/mm2_db/src/indexed_db/indexed_cursor.rs Outdated Show resolved Hide resolved
mm2src/mm2_db/src/indexed_db/indexed_cursor.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

🔥

@shamardy shamardy merged commit 96fac7c into dev Mar 12, 2024
23 of 30 checks passed
@shamardy shamardy deleted the indexedb-cusor-filter-ext branch March 12, 2024 14:29
dimxy added a commit to dimxy/komodo-defi-framework that referenced this pull request Mar 13, 2024
* dev:
  feat(indexeddb): advanced cursor filtering impl (KomodoPlatform#2066)
  update dockerhub destination repository (KomodoPlatform#2082)
  feat(event streaming): configurable worker path, use SharedWorker (KomodoPlatform#2080)
  fix(hd_tests): fix test_hd_utxo_tx_history unit test (KomodoPlatform#2078)
  feat(network): improve efficiency of known peers handling (KomodoPlatform#2074)
  feat(nft): enable eth with non fungible tokens (KomodoPlatform#2049)
  feat(ETH transport & heartbeats): various enhancements/features (KomodoPlatform#2058)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants