Skip to content

Update EW introduction content#2790

Merged
bgravenorst merged 14 commits intomainfrom
DOC-2782
Apr 9, 2026
Merged

Update EW introduction content#2790
bgravenorst merged 14 commits intomainfrom
DOC-2782

Conversation

@bgravenorst
Copy link
Copy Markdown
Contributor

@bgravenorst bgravenorst commented Mar 25, 2026

Description

  • Add a graphic that helps users quickly understand how embedded wallets differ from traditional wallets
  • Updates to remove marketing tone, tighten language, and improve clarity.
  • Move conceptual architecture information into a dedicated topic.
  • Ensured the architecture diagram is presented as a supporting visual, not the primary source of
    information (accessibility issue).
  • Added content to improve description of the architecture content.

Issue(s) fixed

Fixes #2782

Preview

https://metamask-docs-git-doc-2782-consensys-ddffed67.vercel.app/embedded-wallets/

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

Note

Low Risk
Low risk documentation-only changes; main risk is broken links/assets or unintended wording changes affecting technical meaning.

Overview
Updates the Embedded Wallets About page to remove marketing tone, tighten language, and add an onboarding comparison graphic (wallet-user-flow-comparison.svg).

Moves the conceptual architecture explanation and flow diagram into a new dedicated Architecture doc (embedded-wallets/architecture.mdx) and links to it from the About page, with a corresponding sidebar entry.

Adds list-writing guidance to the editorial voice rules (.cursor/rules/editorial-voice.mdc).

Reviewed by Cursor Bugbot for commit cf6562a. Bugbot is set up for automated code reviews on this repo. Configure here.

Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
@bgravenorst bgravenorst requested review from a team as code owners March 25, 2026 03:57
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

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

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Apr 8, 2026 11:13pm

Request Review

Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
yashovardhan
yashovardhan previously approved these changes Apr 8, 2026
Copy link
Copy Markdown
Member

@yashovardhan yashovardhan left a comment

Choose a reason for hiding this comment

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

LGTM

bgravenorst and others added 5 commits April 9, 2026 05:49
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Co-authored-by: Alexandra Carrillo <12214231+alexandratran@users.noreply.github.com>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
@bgravenorst bgravenorst merged commit 917d22c into main Apr 9, 2026
18 checks passed
@bgravenorst bgravenorst deleted the DOC-2782 branch April 9, 2026 04:19
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.

[Docs] Embedded Wallets: clarify traditional vs embedded wallet in intro

3 participants