Skip to content

fix(money): correct Earn on your crypto tooltip copy (MUSD-895)#31074

Merged
Kureev merged 1 commit into
mainfrom
kureev/MUSD-895
Jun 4, 2026
Merged

fix(money): correct Earn on your crypto tooltip copy (MUSD-895)#31074
Kureev merged 1 commit into
mainfrom
kureev/MUSD-895

Conversation

@Kureev

@Kureev Kureev commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Description

The "Earn on your crypto" tooltip bottom sheet displayed incorrect copy. The body text started with "Illustration assumes…", which did not match the approved design copy. This updates the locale string to "Projection assumes…" so the tooltip matches the Figma reference flagged in UAT.

The {{percentage}} interpolation (live APY, fallback 4%) is preserved; only the leading word changed.

Changelog

CHANGELOG entry: null

Related issues

Fixes: MUSD-895

Manual testing steps

Feature: Earn on your crypto tooltip copy

  Scenario: user opens the Earn on your crypto info sheet
    Given user is on the Money Account home screen
    And the "Earn on your crypto" section is visible

    When user taps the info icon next to the "Earn on your crypto" section header
    Then the bottom sheet body reads "Projection assumes 4% Annual Percentage Yield (APY) remains unchanged for one year. APY is variable and is not guaranteed. No guarantee of return."

Screenshots/Recordings

Before

After

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
    • Ideally on a mid-range device; emulator is acceptable
  • I've tested with a power user scenario
    • Use these power-user SRPs to import wallets with many accounts and tokens
  • I've instrumented key operations with Sentry traces for production performance metrics

For performance guidelines and tooling, see the Performance Guide.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Low Risk
Single English string change in locales with no behavioral, security, or data impact.

Overview
Updates the Earn on your crypto info bottom sheet copy in locales/languages/en.json so the body opens with "Projection assumes…" instead of "Illustration assumes…", aligning Money UI text with approved design/UAT (MUSD-895). The {{percentage}} APY placeholder and the rest of the disclaimer sentence are unchanged; no component or logic changes.

Reviewed by Cursor Bugbot for commit 340aa83. Bugbot is set up for automated code reviews on this repo. Configure here.

@Kureev Kureev added team-earn pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. labels Jun 4, 2026
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@Kureev Kureev self-assigned this Jun 4, 2026
@Kureev Kureev removed the pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. label Jun 4, 2026
@Kureev Kureev marked this pull request as ready for review June 4, 2026 14:00
@Kureev Kureev enabled auto-merge June 4, 2026 14:00
@Kureev Kureev added this pull request to the merge queue Jun 4, 2026
Merged via the queue into main with commit 6573e6a Jun 4, 2026
188 of 229 checks passed
@Kureev Kureev deleted the kureev/MUSD-895 branch June 4, 2026 14:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants