Skip to content

fix: instructions on PSA allowlist#7

Open
dchoi27 wants to merge 1 commit intomainfrom
fix-allowlist
Open

fix: instructions on PSA allowlist#7
dchoi27 wants to merge 1 commit intomainfrom
fix-allowlist

Conversation

@dchoi27
Copy link
Copy Markdown

@dchoi27 dchoi27 commented Feb 25, 2022

No description provided.

@dchoi27 dchoi27 requested a review from alanshaw February 25, 2022 15:40
@JeffLowe JeffLowe added the documentation Improvements or additions to documentation label Feb 27, 2022
Copy link
Copy Markdown

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

Left a suggestion to easily direct people to their email client. If thing we should also add a body with the things we want from them already filled in

Comment thread README.md
### Import a list of CIDs

You have a list of CIDs for content you have stored in IPFS already and would like to import that content into nft.storage.
You have a list of CIDs for content you have stored in IPFS already and would like to import that content into NFT.Storage. To be able to do so, you must be added to an allowlist for using the Pinning Service API. If you would like to be added, email `support (at) nft.storage` with your project name and a description, a public profile link for you or your project (Twitter, artist profile, etc.) and 5-10 CIDs of content you're looking to pin.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
You have a list of CIDs for content you have stored in IPFS already and would like to import that content into NFT.Storage. To be able to do so, you must be added to an allowlist for using the Pinning Service API. If you would like to be added, email `support (at) nft.storage` with your project name and a description, a public profile link for you or your project (Twitter, artist profile, etc.) and 5-10 CIDs of content you're looking to pin.
You have a list of CIDs for content you have stored in IPFS already and would like to import that content into NFT.Storage. To be able to do so, you must be added to an allowlist for using the Pinning Service API. If you would like to be added, email [`support (at) nft.storage`](mailto:support@nft.storage?subject=Pinning%20API%20access) with your project name and a description, a public profile link for you or your project (Twitter, artist profile, etc.) and 5-10 CIDs of content you're looking to pin.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We could even add a body with email template with the things we want. For doing that, the URL would look be like:

mailto:support@nft.storage?subject=Pinning%20API%20access&body=Hello

Note that text must be encoded with something like this: https://www.urlencoder.org/

See example https://www.w3docs.com/tools/code-editor/1181

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

(this would likely be cool to add to every place we direct people to sending an email!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants