fix: update http put args description #117
Conversation
vasco-santos
force-pushed
the
fix/update-http-put-args-description
branch
from
b268fde to
5ab1eff
Compare
| // TODO: this should now be updated? | ||
| "address": { "await/ok": { "/": "bafy...alloc" } }, |
There was a problem hiding this comment.
@Gozala this ended up changing below to use url and headers https://github.com/web3-storage/specs/blob/main/w3-blob.md#put-blob-capability-schema
I honestly liked more the previous alignment here with address, but happy to change this if you have strong preference to use url and header top level
There was a problem hiding this comment.
I assume if so, we would like to have something along the lines of
'ucan/await': ['.out.ok.claim', acceptfx.link()],
There was a problem hiding this comment.
I end up updating it here https://github.com/web3-storage/specs/pull/118/files last week but forgot to submit a PR for the review. I do not feel very strongly about it, however I find { url, headers, body } to be more obvious choice for http/put than { address, body }.
|
I think this is superseded by #118 but let me know if you think it should not. |
We updated this but did not change it all