Skip to content

docs: update onramp documentation to reflect Coinbase removal #566

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Jul 14, 2025

docs: update onramp documentation to reflect Coinbase removal

Summary

This PR updates the AppKit onramp documentation to reflect the removal of Coinbase OnRamp functionality from the AppKit SDK. Based on commit 79926ba3de4d7af6ae14a61118ca8c30d183a0ba, Coinbase OnRamp was removed since it's now available under Meld's options, simplifying the onramp provider system.

Changes made:

  • Updated provider selection text from "Select 'Coinbase' or 'Meld' as your provider" to "Select 'Meld' as your provider"
  • Updated purchase completion text to reference only Meld instead of both Coinbase and Meld

Review & Testing Checklist for Human

  • Check if /images/new-onramp.png shows outdated UI - The image may still display Coinbase as an option, which would be inconsistent with the updated text
  • Search for additional Coinbase OnRamp references - Verify no other documentation files (upgrade guides, migration docs, etc.) still reference Coinbase OnRamp
  • Test onramp flow functionality - Manually verify that the onramp flow works correctly with only Meld as the provider
  • Verify consistency across framework docs - Check if similar changes are needed in framework-specific documentation (React, Vue, etc.)

Diagram

%%{ init : { "theme" : "default" }}%%
graph TD
    A["appkit/features/onramp.mdx"]:::major-edit
    B["/images/new-onramp.png"]:::context
    C["AppKit SDK<br/>(commit 79926ba3)"]:::context
    D["Coinbase OnRamp Removal"]:::context
    E["Meld Provider"]:::context
    
    C --> D
    D --> A
    A --> B
    A --> E
    
    subgraph Legend
        L1[Major Edit]:::major-edit
        L2[Minor Edit]:::minor-edit
        L3[Context/No Edit]:::context
    end
    
    classDef major-edit fill:#90EE90
    classDef minor-edit fill:#87CEEB
    classDef context fill:#FFFFFF
Loading

Notes

  • This change aligns documentation with the AppKit SDK changes that removed Coinbase OnRamp functionality
  • Users can still access Coinbase functionality through Meld's integration, so this represents a simplification rather than a feature removal
  • Referenced commit: reown-com/appkit@79926ba
  • Tags: @reown-com/appkit-web-team @reown-com/devrel-team

Link to Devin run: https://app.devin.ai/sessions/6ca8980195164cfeabbc3e9c463d4fc3
Requested by: User via Slack automation

- Remove Coinbase as onramp provider option since it's now available under Meld
- Update provider selection text to only mention Meld
- Update purchase completion text to reference Meld only

References: reown-com/appkit@79926ba

Co-Authored-By: unknown <>
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link

@devin-ai-integration[bot] Please review the tone of voice for the content changes in this PR against Reown's brand guidelines.

📝 Content Review Request

Files to review: 1

  • appkit/features/onramp.mdx

Review focus:

  • Tone alignment with Reown's brand guidelines
  • Clear and accessible language for developers
  • Professional yet approachable communication
  • Consistent terminology usage

Guidelines summary:

  • Clear & Accessible: Translate complex ideas into approachable language
  • Professional yet Friendly: Maintain authority while being welcoming
  • Developer-Focused: Understand technical audience but remain inclusive
  • Avoid: Overly casual language, fear-based messaging, buzzwords, jargon without explanation
  • Embrace: Clear explanations, confident tone, transparency, respectful communication

Please analyze the content changes and provide constructive feedback on tone and voice alignment.

Copy link
Contributor Author

🤖 Tone of Voice Review Response

I've reviewed the content changes in this PR against Reown's brand guidelines:

Changes Made:

  • Line 82: "Select 'Coinbase' or 'Meld' as your provider" → "Select 'Meld' as your provider"
  • Line 90: "Complete your purchase using your selected provider. You will be prompted to sign in to your Coinbase or Meld account" → "Complete your purchase using Meld. You will be prompted to sign in to your Meld account"

Tone Analysis:
Clear & Accessible: The changes maintain simple, direct language that's easy for developers to understand
Professional yet Friendly: The instructional tone remains consistent with existing documentation
Developer-Focused: Changes preserve the step-by-step guidance format developers expect
Consistent Terminology: Uses established terms like 'provider' and 'purchase' consistently

Alignment Assessment:
The content changes align well with Reown's tone guidelines. The modifications:

  • Remove outdated provider options without introducing confusion
  • Maintain the clear, instructional voice of the original content
  • Keep the professional yet approachable tone throughout
  • Preserve the logical flow and structure developers rely on

No tone adjustments needed - the changes successfully update factual content while preserving Reown's established voice.

@bkrem bkrem requested review from a team and Copilot July 15, 2025 09:06
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the AppKit onramp documentation to remove references to Coinbase OnRamp now that Coinbase has been consolidated under Meld.

  • Changed provider selection text to only mention Meld
  • Updated purchase instructions to reference only Meld
  • (Context) The screenshot remains the same but may need updating
Comments suppressed due to low confidence (1)

appkit/features/onramp.mdx:85

  • Verify that /images/new-onramp.png no longer displays Coinbase as an option. If it does, replace it with an updated screenshot showing only Meld.
  <img src="/images/new-onramp.png" />

@@ -79,15 +79,15 @@ Once connected, select "Buy Crypto" from the wallet modal.

### Choose your provider
Copy link
Collaborator

Choose a reason for hiding this comment

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

this page will be deleted
no action need for the moment

Copy link
Contributor Author

Thanks for the heads up @rtomas!

I understand that this entire onramp page will be deleted soon, which would make these changes obsolete. Since you mentioned no action is needed for the moment, I'll leave this PR as-is for now.

For context: This PR was created automatically based on commit 79926ba3de4d7af6ae14a61118ca8c30d183a0ba in the appkit repository, which removed Coinbase OnRamp functionality. The automation detected this as a documentation-impacting change and created this PR to keep docs in sync.

If you'd prefer to close this PR given the planned page deletion, please let me know and I can close it. Otherwise, it can remain open until the page deletion occurs.

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