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

Added list support with prefix functionality - #122

Open
tmayr wants to merge 1 commit into
dollarshaveclub:masterfrom
tmayr:kv-browse-support
Open

tmayr wants to merge 1 commit into
dollarshaveclub:masterfrom
tmayr:kv-browse-support

Conversation

@tmayr

@tmayr tmayr commented Sep 23, 2019 •

Copy link
Copy Markdown

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
Copy Markdown
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

tmayr commented Oct 30, 2019

Copy link
Copy Markdown
Author

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
Copy Markdown

Any chance of this being merged in?

@GregBrimble

GregBrimble commented Mar 27, 2020 •

Copy link
Copy Markdown
Contributor

@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
Copy Markdown

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