storj / docs

Source for Storj docs
https://docs.storj.io
Other
69 stars 34 forks source link

app/dcs/api/s3/s3-compatibility: update with the latest storj.io/gateway/docs/s3-compatibility.md #286

Closed amwolff closed 11 months ago

amwolff commented 1 year ago

This updates the page with the latest storj.io/gateway/docs/s3-compatibility.md to make it the ground truth and prepare for removal of storj.io/gateway/docs/s3-compatibility.md. It also adds another context for GetBucketLocation from https://github.com/storj/docs/pull/285, superseding this pull request.

github-actions[bot] commented 1 year ago

Visit the preview URL for this PR (updated for commit fa4348b):

https://storj-docs--pr286-aw-update-s3-compati-w9evr5f6.web.app

(expires Wed, 18 Oct 2023 11:09:07 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 3c1f0a7f83da2cd749f45c4149016bdb4af1841c

amwolff commented 1 year ago

It looks like the wording for the LocationConstraint section is still pending approval, so better not to merge this without explicit approval from, e.g., @onionjake.

halkyon commented 1 year ago

Do you also want to prepare the change to remove the docs in the gateway repo, so we don't have them in two places any longer?

amwolff commented 12 months ago

I pushed 7400cbd, which fixes the formatting of the tables and consistency of punctuation and capital letters usage after Michael's additions.

onionjake commented 12 months ago

This is good to go as far as I'm concerned. @ferristocrat I think you need to mark it approved because you reviewed with comments?