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

Add the DePIN guide #660

Merged
merged 21 commits into from
Dec 11, 2024
Merged

Conversation

catalinred
Copy link
Contributor

Problem

Missing a DePIN guide in /developers/guides

Summary of Changes

This branch adds the DePIN guide based on recent conversations.

@catalinred catalinred requested a review from ZYJLiu as a code owner December 2, 2024 16:00
@catalinred catalinred requested a review from nickfrosty December 2, 2024 16:01
@catalinred catalinred marked this pull request as draft December 2, 2024 16:19
@catalinred catalinred marked this pull request as ready for review December 3, 2024 14:09
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@nickfrosty nickfrosty left a comment

Choose a reason for hiding this comment

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

suggested some minor updates

content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
consider (discussed
[here](https://solana.stackexchange.com/questions/9205/what-is-the-advantage-of-using-the-token22-token-extensions-program-over-the-old)).
The recommended selection between the two options ultimately reduces to whether
the features in the token extensions program would be of use to the application.
Copy link
Collaborator

Choose a reason for hiding this comment

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

it might be helpful to add some general examples of what some tradeoffs could be, providing context here in the article

content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
#### Automatic Distribution

An alternative to having users proactively claim rewards is automating rewards
distribution. On a regular basis, this “rewards crank” fetches rewards for an
Copy link
Collaborator

Choose a reason for hiding this comment

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

should "this" be a link?

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't have any links for this. We can also change this language to call it a scheduled task instead of a crank, because that term sounds more familiar to me

Copy link
Collaborator

Choose a reason for hiding this comment

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

cranks are common, i think people are familiar enough with it. I just read the paragraph as might be trying to point to a link of something and wanted to make sure we are not missing a link

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't have a link for this, but feel free to change wording via the suggestion feature

content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
content/guides/depin/getting-started.md Outdated Show resolved Hide resolved
@nickfrosty nickfrosty merged commit 2bd171d into solana-foundation:main Dec 11, 2024
2 checks passed
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