Skip to content

Clean up DTK get started pages #2229

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

Merged
merged 6 commits into from
Aug 22, 2025
Merged

Clean up DTK get started pages #2229

merged 6 commits into from
Aug 22, 2025

Conversation

alexandratran
Copy link
Contributor

@alexandratran alexandratran commented Aug 13, 2025

Description

Clean up DTK get started pages as per DTK docs plan:

  • Remove delegation quickstart
  • Move 7702 quickstart as subpage under smart accounts quickstarts
  • Add next steps to all quickstarts
  • Change "CLI quickstart" to "Use the CLI" (to avoid conflation with the other quickstarts)
  • Add redirects and link fixes as needed

Issue(s) fixed

Fixes #2214

Checklist

Complete this checklist before merging your PR:

  • If this PR contains a major change to the documentation content, I have added an entry to the top of the "What's new?" page.
  • The proposed changes have been reviewed and approved by a member of the documentation team.

Copy link

vercel bot commented Aug 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
metamask-docs Ready Ready Preview Comment Aug 22, 2025 2:19am

cursor[bot]

This comment was marked as outdated.

@alexandratran alexandratran changed the title Move DTK CLI instructions to smart account / ERC-7715 quickstarts Clean up DTK get started pages Aug 20, 2025
Copy link
Member

@AyushBherwani1998 AyushBherwani1998 left a comment

Choose a reason for hiding this comment

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

LGTM, do we want to do the delegation changes in separate PR?

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

Some feedback attached

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

lgtm

@alexandratran alexandratran merged commit b4ca1d3 into main Aug 22, 2025
19 checks passed
@alexandratran alexandratran deleted the 2214-split-dtk-cli branch August 22, 2025 02:51
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.

Clean up DTK get started section
3 participants