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

Document "Connect your wallet" for asset hub smart contracts #300

Open
wants to merge 10 commits into
base: staging-ah-smart-contracts
Choose a base branch
from

Conversation

nhussein11
Copy link
Collaborator

@nhussein11 nhussein11 commented Jan 7, 2025

This PR aims to add a basic documentation of how to connect your wallet to asset hub smart contracts

Note: The IA used is preliminary. We can circle back on this later on and define where all this content should be located

@nhussein11 nhussein11 requested a review from a team as a code owner January 7, 2025 14:16
@nhussein11 nhussein11 linked an issue Jan 7, 2025 that may be closed by this pull request
@nhussein11 nhussein11 changed the title Document Connect your wallet for asset hub smart contracts Document "Connect your wallet" for asset hub smart contracts Jan 7, 2025
@nhussein11 nhussein11 changed the base branch from master to staging-ah-smart-contracts January 7, 2025 15:29
Copy link
Collaborator

@CrackTheCode016 CrackTheCode016 left a comment

Choose a reason for hiding this comment

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

overall lgtm

develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved
develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved
develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved

To get started with Metamask, you need to install the [MetaMask extension](https://metamask.io/download/){target=\_blank} and add it to the browser. Once, you have metamask installed, you can set up a new wallet and securely store your seed phrase. This phrase is crucial for recovery in case you lose access.

## Connect to the Asset Hub Westend Testnet
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think there might be a button / embed we could put here too, see: https://contracts.polkadot.io/connect-to-asset-hub/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I thought the https://contracts.polkadot.io/connect-to-asset-hub/ was a temporary solution until this documentation is ready. If not, we can add the link at the end of this page to connect to contracts.polkadot.io. Do you know if that is the case?

develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved
develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved
develop/smart-contracts/evm/connect-your-wallet.md Outdated Show resolved Hide resolved
@nhussein11 nhussein11 requested a review from a team January 7, 2025 17:39

## Connect to Asset Hub Using MetaMask

MetaMask allows users to connect to the Asset Hub to explore and interact with the chain. This article will guide you step by step on how to set up an EVM-compatible wallet, connect to the Westend Asset Hub, and request test tokens for development testing.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not accurate

  1. Let's start with what is Westend --> one of the Polkadot test networks.
  2. MetaMask is not just to connect to Westend, it can connect to any EVM compat chain
  3. Westend is not EVM compat, and soon Kusama and Polkadot will be
  4. "explore and interact with the chain" is not correct, you can only interact with smart contracts. You cannot do a balance transfer in Westend with MetaMastk.
  5. In fact, it is important to explore how a user should use a Polkadot-native wallet with Westend, and how MetaMastk, and how they differ.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, @kianenigma. I assume there’s a typo in item 2, and you meant to say, "Westend is EVM compatible, and soon Kusama and Polkadot will be."

If so, do you think this introduction covers your points using the correct terminology?

MetaMask is a popular wallet for interacting with EVM-compatible chains, allowing users to connect to test networks that support Ethereum-based smart contracts. This article provides guidance on setting up MetaMask, connecting to the contracts deployed on the EVM-compatible test network, the Westend Asset Hub, and requesting test tokens for development purposes.

It’s important to emphasize that MetaMask primarily facilitates interactions with smart contracts, giving you access to various functionalities available on the chain.

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

Of course the content here is okay, but I am having a hard time approving it, as I don't know where it sits in the IA.

@nhussein11
Copy link
Collaborator Author

nhussein11 commented Jan 8, 2025

Of course the content here is okay, but I am having a hard time approving it, as I don't know where it sits in the IA.

Thanks, this page will be merged into staging-ah-smart-contracts, which is a branch containing the initial IA where we intended to place this content.

For this particular page, the path would be: /develop/smart-contracts/evm-toolkit/connect-your-wallet.md

@nhussein11 nhussein11 added the B0 - Needs Review Pull request is ready for review label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0 - Needs Review Pull request is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document how to connect Metamask to Westend AssetHub
3 participants