Skip to content

feat: list kv store keys by collection of prefix #688

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 2 commits into from
May 16, 2025

Conversation

MFori
Copy link
Member

@MFori MFori commented May 2, 2025

@MFori MFori added the t-console Issues with this label are in the ownership of the console team. label May 2, 2025
@MFori MFori self-assigned this May 2, 2025
@github-actions github-actions bot added this to the 113th sprint - Console team milestone May 2, 2025
@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label May 16, 2025
@MFori MFori requested review from Copilot and barjin May 16, 2025 09:16
@MFori MFori marked this pull request as ready for review May 16, 2025 09:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for filtering key-value store keys by adding new query parameters ("collection" and "prefix"). The changes include:

  • Updating the test case to pass the new query parameters.
  • Adding validation for the new parameters in the client and updating the TypeScript interface.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
test/key_value_stores.test.js Updated test query to include new "collection" and "prefix" options
src/resource_clients/key_value_store.ts Added validation and interface properties for "collection" and "prefix" parameters

Copy link
Contributor

@barjin barjin left a comment

Choose a reason for hiding this comment

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

Looks pretty straightforward to me, thanks! 🚀

@barjin
Copy link
Contributor

barjin commented May 16, 2025

Also, I suppose we should implement this in MemoryStorage in Crawlee too (at least the prefix param) for parity with Platform-based KVS.

Issue in Crawlee here

@MFori MFori merged commit be25137 into master May 16, 2025
7 checks passed
@MFori MFori deleted the feat/load-keys-by-collection branch May 16, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-console Issues with this label are in the ownership of the console team. tested Temporary label used only programatically for some analytics.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants