fix(settings): update notifications settings copy and casing

[andrewjcohen]

Description

Updates copy in the Notifications settings screen to match current content guidelines. Section titles now use sentence case (e.g. "Wallet activity" instead of "Wallet Activity"), the wallet activity description no longer mentions rewards, and the "curated" typo in the trading signals description is fixed. Related unit tests are updated to match the new strings.

Changelog

CHANGELOG entry: Updated notification settings section titles and descriptions

Related issues

Refs: content update — no linked issue

Manual testing steps

Feature: Notifications settings copy

  Scenario: user views updated notification section labels
    Given the user has notifications enabled in Settings
    And the app language is set to English

    When user navigates to Settings > Notifications
    Then section titles display sentence case (e.g. "Wallet activity", "Trading activity", "Price alerts")
    And the wallet activity description reads "Buy, sells, transfers, swaps"
    And the trading signals description reads "Updates from traders and assets you follow, plus curated market news"

Screenshots/Recordings

N/A — copy-only change with no visual layout differences.

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

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
  • See trace() for usage and addToken for an example

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.

Made with Cursor

[github-actions]

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.

[mm-token-exchange-service]

PR template — items to address before "Ready for review"

Warnings — informational, address before merging:

• Pre-merge author checklist has unchecked items (e.g. "I've documented my code using JSDoc format if applicable"). Every box must be consciously checked — see docs/readme/ready-for-review.md.

See docs/readme/ready-for-review.md for the full Definition of Ready for Review.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 102f308. Configure here.}

[cursor]

Stale socialAI title in test

Medium Severity

This commit changes social_ai_title in en.json to Trading signals, and TopTradersView passes that locale value into navigation params. The socialAI notification navigation test still expects Trading Signals, so mockNavigate assertions no longer match runtime behavior and the test fails.

 

^{Reviewed by Cursor Bugbot for commit 102f308. Configure here.}