Conversation
|
| @@ -0,0 +1,128 @@ | |||
| { | |||
| "aggregators": { | |||
There was a problem hiding this comment.
Question: Are you using an object where the URLs are the keys to make it extensible in the future? E.g., adding info about who runs it, its location, etc.?
| "https://walrus-testnet.validators.services.kyve.network/publish": {}, | ||
| "http://walrus-publisher-testnet.suisec.tech:9001": {} | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Nit: Can we add a final new line?
| } | |
| } | |
|
|
||
| The field `event` returns the [Sui event ID](../dev-guide/sui-struct.md) that can be used to | ||
| find the transaction that created the Sui Blob object on the Sui explorer or using a Sui SDK. | ||
|
|
||
| ### Read | ||
|
|
||
| Blobs may be read from an aggregator or daemon using HTTP GET. For example, the following cURL | ||
| command reads a blob and writes it to an output file: | ||
|
|
||
| ```sh | ||
| curl "$AGGREGATOR/v1/blobs/<some blob ID>" -o <some file name> | ||
| ``` | ||
|
|
||
| Alternatively you may print the contents of a blob in the terminal with the cURL command: | ||
|
|
||
| ```sh | ||
| curl "$AGGREGATOR/v1/blobs/<some blob ID>" | ||
| ``` | ||
|
|
||
| ```admonish tip title="Content sniffing" | ||
| Modern browsers will attempt to sniff the content type for such resources, and will generally do a | ||
| good job of inferring content types for media. However, the aggregator on purpose prevents such | ||
| sniffing from inferring dangerous executable types such as JavaScript or style sheet types. | ||
| ``` |
There was a problem hiding this comment.
Question: Why is this removed?
.github/actions/deploy/deploy.yaml
Outdated
There was a problem hiding this comment.
Question: Why is this needed? We already deploy the docs with the publish.yaml workflow.
There was a problem hiding this comment.
Ok! Didn't know that! Will remove
| cargo install mdbook-admonish@1.18.0 --locked | ||
| cargo install mdbook-katex@0.9.0 --locked | ||
| cargo install mdbook-i18n-helpers --locked | ||
| cargo install mdbook-variables |
There was a problem hiding this comment.
Important: This will probably have to be added to the build-mdbook action.
There was a problem hiding this comment.
Currently, the preview looks like this:
There was a problem hiding this comment.
Question: Where/how is this used?
.github/actions/deploy/deploy.yaml
Outdated
There was a problem hiding this comment.
Ok! Didn't know that! Will remove
|
Closed in favor of #293 |
2 participants
No description provided.