Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

feat create namespace #392

Merged
merged 3 commits into from
Aug 6, 2019
Merged

Conversation

ashleymichal
Copy link
Contributor

@ashleymichal ashleymichal commented Aug 3, 2019

Closes #340 . Includes commit from #391 (merge first).

This is dependent on an update to the cloudflare-rs library per this issue: cloudflare/cloudflare-rs#10.

Interacting with this code looks a little something like this:
On successful creation:

$ wrangler kv create "my namespace"
🌀  Creating namespace with title "my namespace"
✨  Success: WorkersKVNamespace {
    id: "f7b02e7fc70443149ac906dd81ec1791",
    title: "my namespace",
}

On API error (in this case duplicate kv namespace):

$ wrangler kv create "my namespace"
🌀  Creating namespace with title "my namespace"
⚠️  Error 10014: a namespace with this account ID and title already exists
🕵️‍♂️  Run `wrangler kv list` to see your existing namespaces with IDs

@ashleymichal ashleymichal added changelog - feature regression Something is broken, but works in previous releases labels Aug 3, 2019
@ashleymichal ashleymichal force-pushed the alewis/feat-create-namespace branch 5 times, most recently from 4473130 to 718f67f Compare August 3, 2019 22:28
@ashleymichal ashleymichal added this to the Add KV Subcommands milestone Aug 3, 2019
* implement From for cloudflare-rs Credential -> GlobalUser
Cargo.toml Show resolved Hide resolved
src/commands/kv/mod.rs Show resolved Hide resolved
src/main.rs Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
src/terminal/emoji.rs Show resolved Hide resolved
src/main.rs Show resolved Hide resolved
@ashleymichal ashleymichal merged commit 374860d into feat-kv-commands Aug 6, 2019
@delete-merged-branch delete-merged-branch bot deleted the alewis/feat-create-namespace branch August 6, 2019 14:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
regression Something is broken, but works in previous releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants