Skip to content

Update CI workflow and Embedded Wallets SDK documentation#2753

Open
Audbre6 wants to merge 1 commit intoMetaMask:mainfrom
Audbre6:patch-5
Open

Update CI workflow and Embedded Wallets SDK documentation#2753
Audbre6 wants to merge 1 commit intoMetaMask:mainfrom
Audbre6:patch-5

Conversation

@Audbre6
Copy link

@Audbre6 Audbre6 commented Mar 11, 2026

Updated the documentation for the Embedded Wallets SDK, including setup instructions and environment variable configuration.

Description

Issue(s) fixed

Fixes #

Preview

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.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Low Risk
Low code risk since this is documentation-only, but the inserted 0x... strings and raw <img> tags may break markdown rendering and confuse readers.

Overview
Updates sdk/connect/javascript-web3auth.md with changes to the Embedded Wallets (Web3Auth) JavaScript quickstart, including additional inline content and screenshots.

The diff also introduces repeated 0xED699... strings throughout headings/code blocks and injects raw <img> tags inside a TypeScript snippet, which likely breaks formatting and copy/paste usability of the guide.

Written by Cursor Bugbot for commit 76e0239. This will update automatically on new commits. Configure here.

Updated the documentation for the Embedded Wallets SDK, including setup instructions and environment variable configuration.
@Audbre6 Audbre6 requested review from a team as code owners March 11, 2026 00:18
@vercel
Copy link

vercel bot commented Mar 11, 2026

@Audbre6 is attempting to deploy a commit to the Consensys Team on Vercel.

A member of the Team first needs to authorize it.

@Audbre6
Copy link
Author

Audbre6 commented Mar 11, 2026

Copy link

@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 2 potential issues.

@@ -1,22 +1,22 @@
---
---0xED699044C103691c01dADb60EbC9980Bfb9Ea906
Copy link

Choose a reason for hiding this comment

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

Ethereum address string littered throughout documentation file

High Severity

The string 0xED699044C103691c01dADb60EbC9980Bfb9Ea906 (an Ethereum address) has been accidentally appended to ~35 lines throughout the documentation file. It corrupts the YAML frontmatter on line 1 (---0xED699044C...), breaks HTML tags, renders as visible gibberish throughout the page, and breaks markdown heading anchors (e.g. ## Set up manually0xED699...). This will cause rendering issues across the entire Embedded Wallets SDK quickstart page.

Additional Locations (2)
Fix in Cursor Fix in Web

export const Navbar = () => {
const { connect } = useWeb3AuthConnect();
<img width="1080" height="2400" alt="40666" src="https://github.com/user-attachments/assets/a5fc24f6-90fb-4bb9-a4d7-aa860c676afb" />
<img width="787" height="1233" alt="40665" src="https://github.com/user-attachments/assets/83ce4fd1-c977-401c-9594-8998a045e3ac" />
Copy link

Choose a reason for hiding this comment

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

Screenshot images accidentally pasted inside code block

High Severity

Two <img> HTML tags with GitHub user-attachment URLs have been accidentally inserted inside a TypeScript code block, between the useWeb3AuthConnect hook call and the return statement. These are likely screenshots that were pasted into the PR description or comments and ended up in the code. They will render as raw HTML text inside the code example, confusing readers.

Fix in Cursor Fix in Web

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.

1 participant