Public
Val Town is a social website to write and deploy JavaScript.
Build APIs and schedule functions from your browser.
Comments
8
davidcrespo avatar

I think it would be helpful if this doc mentioned the scope of the blob storage. My assumption, because it is not mentioned, is that this is global (account-scoped), not val-scoped, i.e., if I get and set the same key from two different vals, they will be sharing and overwriting the same value in the global store.

std avatar

Great note! I added this to the readme above: "Blob storage is scoped globally to your account. If you set a blob in one val, you can retrieve it by the same key in another val." What do you think?

davidcrespo avatar

Sounds good!

davidcrespo avatar

Another note: I converted a val to use blob instead of separate val to store a string value, and I expected to use get and set because set has value: string | BodyInit for that. Took me a bit to figure out that

  1. get returns a Response (though I can see why it does) and it is not what you want for storing a string
  2. I can use getJSON with a plain string because a string is valid JSON

I think of these as things that could use noting in the docs rather than a problem with the API. However, I will say unqualified get/set presents to me as the default/simplest API unless I'm told otherwise, but it seems like getJSON/setJSON are actually the first thing most people will want to reach for. That is hinted at by the fact that most of the examples on this page and here use the JSON variant. I think get/set are good names based on the implementation (get is most general, getJSON wraps get) but it would be nice to see that more clearly in the docs.

std avatar

Great point! I reorganized the readme to emphasize how they are lower-level primitives. Lmk if you have any other suggestions

pomdtr avatar

It would be nice to have a rename/move function here.

stevekrouse avatar

I don't think rename/move is in the s3 API, so it'd be a get and then set operation, in which case building it would be a great pull request to this val (when we launch pull requests later today or tomorrow)

std avatar

Thanks for the PR @pomdtr!

Updated: August 13, 2024