[Delegation Toolkit] Add MetaMask Smart Accounts quickstart#2132
[Delegation Toolkit] Add MetaMask Smart Accounts quickstart#2132AyushBherwani1998 merged 4 commits intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
alexandratran
left a comment
alexandratran
left a comment
There was a problem hiding this comment.
Added minor suggestions. If we are introducing "MetaMask Smart Account" branding here, I think we should keep it consistent throughout the page and also make consistent changes across the rest of the DTK docs. I will work on a PR for the latter today.
| import { createPublicClient, http } from "viem"; | ||
| import { sepolia as chain } from "viem/chains"; | ||
| import { createPublicClient, http } from 'viem' |
There was a problem hiding this comment.
| import { createPublicClient, http } from 'viem' | |
| import { createPublicClient, http } from "viem" |
We try to use double quotes consistently throughout the doc site. This applies to the rest of the code samples on this page.
There was a problem hiding this comment.
I think it's my habit of saving files frequently, and since Prettier is configured to use single quotes, it keeps formatting them that way. Maybe we should consider changing the config?
Prettier configuration: https://github.com/MetaMask/metamask-docs/blob/main/.prettierrc
|
@AyushBherwani1998 Here's a follow up PR: #2134 |
3 participants
Description
Adds quickstart for MetaMask Smart Accounts
Issue(s) fixed
Fixes #
Preview
Checklist
Complete this checklist before merging your PR: