Skip to content

feat: add interactive payment demo to accounts SDK getting started#238

Merged
brendanjryan merged 1 commit intomainfrom
brendan/accounts-demo
Apr 7, 2026
Merged

feat: add interactive payment demo to accounts SDK getting started#238
brendanjryan merged 1 commit intomainfrom
brendan/accounts-demo

Conversation

@brendanjryan
Copy link
Copy Markdown
Contributor

@brendanjryan brendanjryan commented Apr 7, 2026

Replaces the standalone sign-in button on the Accounts SDK getting started page with a full interactive demo (Connect → Add Funds → Send Payment), matching the demo on the send-a-payment guide.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 7, 2026 7:13pm
tempo-docs Ready Ready Preview, Comment Apr 7, 2026 7:13pm

Request Review

@brendanjryan brendanjryan force-pushed the brendan/accounts-demo branch 2 times, most recently from 682c86f to 4474162 Compare April 7, 2026 18:07
@chatgpt-codex-connector
Copy link
Copy Markdown

💡 Codex Review

https://github.com/tempoxyz/docs/blob/287e72c56354a5c74b5611b91bd9ed20cb14fc84/src/components/lib/wallets.ts#L9
P2 Badge Filter out tempoWallet from injected connectors

After adding tempoWallet to the default Wagmi config, this filter still excludes only webAuthn, so xyz.tempo is now treated as an injected browser wallet. Components that rely on this helper (e.g. wallet-connection flows under multiInjectedProviderDiscovery) will show Tempo Wallet as a browser-wallet option and can satisfy “connect external wallet” steps without an actual injected wallet connection. Exclude xyz.tempo here alongside other non-injected connectors.


https://github.com/tempoxyz/docs/blob/287e72c56354a5c74b5611b91bd9ed20cb14fc84/src/pages/guide/use-accounts/embed-tempo-wallet.mdx#L1-L3
P2 Badge Add required title frontmatter for SEO metadata

This new page frontmatter includes description but omits title, which violates the repo’s page requirements and leaves SEO/Open Graph title metadata incomplete for this route. Since the docs config uses frontmatter title/description for page metadata and OG generation, add a title field to avoid degraded metadata on /guide/use-accounts/embed-tempo-wallet.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@vercel vercel bot temporarily deployed to Preview – tempo-docs April 7, 2026 18:11 Inactive
@brendanjryan brendanjryan force-pushed the brendan/accounts-demo branch from 4474162 to abd7148 Compare April 7, 2026 18:14
@brendanjryan brendanjryan merged commit 2228fd7 into main Apr 7, 2026
4 of 9 checks passed
@brendanjryan brendanjryan deleted the brendan/accounts-demo branch April 7, 2026 18:15
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