Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Added list support with prefix functionality #122

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tmayr
Copy link

@tmayr tmayr commented Sep 23, 2019

Added support for list() and kv.list({ prefix }) as per the cf docs

Let me know if any changes are needed.

EDIT: Updated code, it's list not browse

@tmayr tmayr changed the title Added browse support with prefix functionality Added list support with prefix functionality Sep 24, 2019
@GregBrimble
Copy link
Contributor

Hey, sorry I never saw your PR when I filed #130. I've got an implementation of cursor, and list_complete over there. Do you think we could combine them together in this PR?

@tmayr
Copy link
Author

tmayr commented Oct 30, 2019

Hey @GregBrimble

Yeah sure thing!, you can make PR on my branch I guess? Or I can make the changes, lmk what you prefer.

@travis-r6s
Copy link

Any chance of this being merged in?

@GregBrimble
Copy link
Contributor

GregBrimble commented Mar 27, 2020

@tmayr , apologies, I totally missed your reply back in October.

@thetre97 , I think this repo has been abandoned now (see the README.md), but Cloudflare's wrangler tool now has the dev command, which lets you locally test a worker. Personally, I have been using it instead of cloudworker for a while now.

@travis-r6s
Copy link

Ah, I had seen the preview command, but not the new dev command. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants