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

Verify Discord Role Instructions #511

Merged
merged 3 commits into from May 13, 2024
Merged

Verify Discord Role Instructions #511

merged 3 commits into from May 13, 2024

Conversation

ryyeung
Copy link
Contributor

@ryyeung ryyeung commented May 9, 2024

No description provided.

Copy link

netlify bot commented May 9, 2024

Deploy Preview for subspace-docs-staging ready!

Name Link
🔨 Latest commit 98729a1
🔍 Latest deploy log https://app.netlify.com/sites/subspace-docs-staging/deploys/66429ccf923f170008d3a772
😎 Deploy Preview https://deploy-preview-511--subspace-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@EmilFattakhov EmilFattakhov left a comment

Choose a reason for hiding this comment

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

That's a good start, thanks Ryan!

A few changes required imho:

  1. This should be under the "Additional Guide" section, and not a top level article.
  2. For the screenshot highlighting - use some software that draws the ovals for you so you couldn't see the strokes.
  3. Need to make few additional screenshots with what the section to claim the role looks like, what is the authorization process, how do you sign the transaction and what are pop up windows you should expect (e.g. authorize discord pop-up window).
  4. We should also add a section about "What if I don't see my wallet in the dropdown many" - something we've discussed on a daily earlier today. You can mimic that by creating a few subwallets locally.

Copy link
Member

@jim-counter jim-counter left a comment

Choose a reason for hiding this comment

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

Can we move it under "Addiitional Guides" please Ryan?

@@ -0,0 +1,8 @@
{
"label": "Verfiy Your Farmer",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"label": "Verfiy Your Farmer",
"label": "Verify Your Farmer",

"position": 3,
"link": {
"type": "generated-index",
"description": "Verfiy Your Farmer Through Discord!"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"description": "Verfiy Your Farmer Through Discord!"
"description": "Verify Your Farmer Through Discord!"

@ryyeung
Copy link
Contributor Author

ryyeung commented May 12, 2024

@EmilFattakhov Added the screenshots. The sizes of the images make it difficult to format, lmk if you have any tips for this.

Copy link
Contributor

@EmilFattakhov EmilFattakhov left a comment

Choose a reason for hiding this comment

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

Thanks Ryan, looks much better, almost there!


## Eligibility for the Verified Farmer Role

**If you have earned a block or vote reward**, you can now link your Discord account to your farmer wallet through the Astral application to obtain this role. Here’s how to do it:
Copy link
Contributor

Choose a reason for hiding this comment

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

Let’s rename “Astral Application” to just “Astral” and make a link, please

2. Sign into your wallet through Subwallet or Polkadot.
![Select Wallet](/img/doc-imgs/verify-farmer/select-wallet.png)

3. Find the new "Connect Discord" section within the panel.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let’s use the name of the section from the screenshot instead of “Connect Discord”.

Copy link
Contributor

Choose a reason for hiding this comment

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

It should be “Get your Farmer role on Discord”

![Verify Wallet](/img/doc-imgs/verify-farmer/verify-wallet.png)
![Successful Verify](/img/doc-imgs/verify-farmer/successful-verify.png)
<br/><br/>
5. Select **Join our Discord Server** and **Connect your Discord Account!**.
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
5. Select **Join our Discord Server** and **Connect your Discord Account!**.
5. Select **Join our Discord Server** and **Connect your Discord Account**!
Suggested change
5. Select **Join our Discord Server** and **Connect your Discord Account!**.
5. Select **Join our Discord Server** and **Connect your Discord Account!**.

@ryyeung ryyeung merged commit 91e5158 into main May 13, 2024
5 checks passed
@ryyeung ryyeung deleted the verified_farmer branch May 13, 2024 23:10
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.

None yet

4 participants