Skip to content

Conversation

Jnix2007
Copy link
Contributor

summary

replaced the placeholder "coming soon" content in docs/base-account/framework-integrations/cdp.mdx with a comprehensive integration guide

updated content

  • complete step-by-step implementation guide
  • working code snippets for React hooks and components
  • proper provider setup using CDPReactProvider + CDPHooksProvider

so what?

  • removes outdated CDP Wagmi references
  • uses CDP React provider approach for wallet persistence
  • includes critical CORS config guidance
  • provides production-ready code snippets

but did you test it?

built & tested a complete demo app following this guide to ensure all steps work correctly

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Sep 29, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@Jnix2007 Jnix2007 force-pushed the feat/cdp-base-account-integration-guide branch from de65e22 to 19ea33f Compare September 29, 2025 15:26
@youssefea youssefea merged commit 4a918dc into base:master Sep 29, 2025
3 checks passed
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.

3 participants