Skip to content

docs: update Coinbase banner to success state in Pay with Exchange page#751

Open
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1772018390-coinbase-pwe-update
Open

docs: update Coinbase banner to success state in Pay with Exchange page#751
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin/1772018390-coinbase-pwe-update

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Feb 25, 2026

Description

Updates the Coinbase availability banners on the Pay with Exchange page. All three <Warning> banners stating "Coinbase is currently not available" have been replaced with <Check> (green/success) banners stating "Coinbase is now working with the Pay with Exchange flow."

Each banner is timestamped with "Update - February 2026" in bold.

The three locations updated are:

  1. Top of the page (intro section)
  2. "Networks and Assets Supported" section
  3. FAQ accordion ("Which exchanges are supported?")

Human Review Checklist

  • Verify the <Check> component renders as a green success banner (not just a neutral callout)
  • Verify bold markdown (**Update - February 2026:**) renders correctly inside the <Check> component
  • Consider whether all 3 banners should remain or if some can be removed now that Coinbase is fully available
  • Confirm the wording "Coinbase is now working with the Pay with Exchange flow" is the desired final copy

Tests

  • Ran the changes locally with Mintlify and confirmed that the changes appear as expected.
  • Ran a grammar check on the updated/created content using ChatGPT.

Direct link to the deployed preview files


Link to Devin run: https://app.devin.ai/sessions/84ffccb1bc7645d4b86899c7889da498
Requested by: @sandiforward

@devin-ai-integration
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

@claude
Copy link

claude bot commented Feb 25, 2026

Claude finished @devin-ai-integration[bot]'s task in 1m 4s —— View job


Code Review

  • Read CLAUDE.md and tone of voice guide
  • Review changed file
  • Provide feedback

Found 2 issue(s)

Issue 1: Redundant duplicate banners — consider removing 2 of the 3

ID: pay-with-exchange-redundant-check-banners-a3f1
File: appkit/payments/pay-with-exchange.mdx:48–50, 115–117
Severity: LOW
Category: code_quality

Context:

  • Pattern: Three identical <Check> banners with the same text appear across the page — intro (line 7), Networks section (line 48), and FAQ accordion (line 115).
  • Risk: The page intro already mentions Coinbase by name; the Networks section lists Coinbase assets explicitly; the FAQ accordion text already reads "Coinbase and Binance." A status banner that repeats in every section adds noise rather than clarity.
  • Impact: Readers who already saw the banner at the top will encounter two more identical callouts, which dilutes the signal value of the <Check> component.
  • Trigger: Any user reading the full page sequentially.

Recommendation: Keep the intro banner (line 7–9) as the canonical status signal. Remove the duplicate in the Networks section (lines 48–50) — the Coinbase asset list directly below it already implies availability. Remove or simplify the one inside the FAQ accordion (lines 115–117) since the accordion text already states Coinbase is supported. Fix this →


Issue 2: Wording "now working" is transitional — conflicts with Reown's confident voice

ID: pay-with-exchange-tone-now-working-b8c2
File: appkit/payments/pay-with-exchange.mdx:8, 49, 116
Severity: LOW
Category: code_quality

Context:

  • Pattern: "Coinbase is now working with the Pay with Exchange flow." uses "now working" — passive, transitional phrasing that implies a recent fix rather than stable support.
  • Risk: Per the tone of voice guide, Reown's voice is confident and not hedging. "Now working" reads as a status update, not a capability statement. Over time it will also read awkwardly since "now" is time-sensitive.
  • Impact: Minor tone inconsistency; page-level only, not brand-critical.
  • Trigger: Whenever this page is read by developers evaluating Coinbase support.

Recommendation: Replace with confident, present-tense language: Coinbase is fully supported in the Pay with Exchange flow. This aligns with the tone guide's "confident but approachable" principle and avoids implying a recent/temporary state. Fix this →

Copy link
Contributor Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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