Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add terms of service #8

Merged
merged 7 commits into from
Nov 14, 2023
Merged

docs: add terms of service #8

merged 7 commits into from
Nov 14, 2023

Conversation

heyjay44
Copy link
Contributor

@heyjay44 heyjay44 commented Nov 13, 2023

  • add two links (line 2 and line 26)
  • update date (line 2)
  • check formatting
  • check links
  • need to move .md file to correct part of the directory tree - >src/pages/docs/terms.md
  • Figure out plans links

Copy link
Contributor

github-actions bot commented Nov 13, 2023

@olizilla olizilla changed the title docs/add-tos docs: add terms of service Nov 13, 2023
src/terms Outdated
@@ -0,0 +1,89 @@
# **[Terms of service](https://web3.storage/terms/#terms-of-service)**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let headings be headings

Suggested change
# **[Terms of service](https://web3.storage/terms/#terms-of-service)**
# Terms of service

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree but I'm not sure if this is link is used elsewhere to reference/direct to this section.

src/terms Outdated Show resolved Hide resolved
src/terms Outdated

Nitro Data LLC (“we,” “us”) may make changes to this Agreement and pricing from time to time. If you do not agree to these revisions, you may stop using the Services.

#### **[Storage term](https://web3.storage/terms/#storage-term)**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#### **[Storage term](https://web3.storage/terms/#storage-term)**
## Storage term

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above.

src/terms Outdated Show resolved Hide resolved
src/terms Outdated

#### **Paid Accounts**

<u>Billing. </u>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why underscore billing? can use an h3?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just copying the formatting from the document that was approved.

src/terms Outdated

You agree that you’ll pay the amounts set forth in https://web3.storage/account/payment/ for your chosen tier (“Subscription Charge”) plus any charges for additional usage beyond the storage limit of your selected tier during the previous month (“Overages”) (collectively, “Subscription Charge” and “Overages” are referred to as (the “Fees”) for each billing cycle).

When determining tier storage limits and Overages, we track usage for payment using the volume of data associated with shard content identifiers/CIDs (CIDs referencing the actual data sent to web3.storage and stored) – usage is not tracked based on content CIDs (CIDs used to reference and access uploads in a format useful to users, like specific files and directories). See more detail on this point here(ADD LINK).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ADD LINK

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dchoi27 which reference link did you have in mind?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah good catch. it should be a link to a section of docs explaining store vs upload. we can remove for launch if too much to coordinate and put back after

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am removing for now until we have docs for this, maybe a FAQ

src/terms Outdated

You authorize us to charge any Payment Method associated with your account in case your primary Payment Method is declined or no longer available to us for payment of Fees you owe us. You remain responsible for any uncollected Fees. If a payment is not successfully settled, due to expiration, insufficient funds, or otherwise, and you do not cancel your account, we may suspend your access to the Paid Plan until we have successfully charged a valid Payment Method. For some Payment Methods, the issuer may charge you additional fees, such as foreign transaction fees or other fees relating to the processing of your Payment Method.

<u>Cancellation. </u>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these all look like h3 items

Suggested change
<u>Cancellation. </u>
### Cancellation.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, copying format from received document. It was underlined.

src/terms Outdated Show resolved Hide resolved
Copy link
Contributor

@olizilla olizilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should live at src/pages/docs/terms.md (needs .md extention)
headers need fixing. links and dates filling out.

@olizilla
Copy link
Contributor

@heyjay44 are you using a tool to make the markdown? it's got the same formatting errors as the docs content?

@vasco-santos vasco-santos force-pushed the docs/add-tos-privacy branch 2 times, most recently from 191929b to e16874f Compare November 14, 2023 07:27
@heyjay44
Copy link
Contributor Author

@olizilla I'm just using vscode and the formatting from the previous ToS.

src/pages/docs/terms.md Outdated Show resolved Hide resolved
src/pages/docs/terms.md Outdated Show resolved Hide resolved
src/pages/docs/terms.md Outdated Show resolved Hide resolved
@vasco-santos vasco-santos merged commit fa286ce into main Nov 14, 2023
5 checks passed
@vasco-santos vasco-santos deleted the docs/add-tos-privacy branch November 14, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants