Skip to content

feat(home): update welcome text for improved clarity#4932

Open
XyneSpaces wants to merge 1 commit into
mainfrom
feat/home-welcome-text-v2
Open

feat(home): update welcome text for improved clarity#4932
XyneSpaces wants to merge 1 commit into
mainfrom
feat/home-welcome-text-v2

Conversation

@XyneSpaces

Copy link
Copy Markdown
Collaborator

Type of Change

  • Text Update / Copy Improvement

Description

Updated the home page welcome text to be more concise and action-oriented.

Changes

Before:

Welcome to the home of your Payments Control Center. It aims to provide your team with a 360-degree view of payments.

After:

Welcome to your Payments Control Center — one place for your team to track and manage every payment

Files Modified

  • src/screens/Home/Home.res — Updated subTitle prop in PageHeading component (line 22)
  • playwright-tests/support/pages/homepage/HomePage.ts — Updated test locator
  • playwright-tests/support/commands.ts — Updated test assertion

Build Status

✅ ReScript compilation: successful
✅ Production build: successful

Testing

Manual testing performed with production API configuration. Screenshots and video recording captured for visual verification.

Note: Before/after screenshots are being prepared and will be added to this PR shortly.

@XyneSpaces XyneSpaces requested a review from a team as a code owner June 5, 2026 16:45
@semanticdiff-com

semanticdiff-com Bot commented Jun 5, 2026

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  playwright-tests/support/commands.ts  51% smaller
  playwright-tests/support/pages/homepage/HomePage.ts  51% smaller
  src/screens/Home/Home.res Unsupported file format

@XyneSpaces XyneSpaces left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Review Summary

Classification: UI Text Update — Home Page Welcome Message
Risk Level: Very Low (copy change only, no functional code)
Files Changed: 3 files (1 source, 2 test files)


Verification Against Control Center Checklist

Tier 1 — High Signal:

Naming Consistency — N/A (no new identifiers introduced)
Test Coverage Maintained — Playwright test locators and assertions updated in sync with source change

Tier 2 — Medium Signal:

String Consistency — New text follows sentence case convention
No Magic Values — Text is appropriately inline for single-use UI copy

Tier 3 — Lower Signal:

Typography Usage — Uses existing body.lg.medium pattern already present in file


Findings

✅ Clean propagation to tests

Both Playwright files updated correctly:

  • HomePage.ts — page object locator updated
  • commands.ts — login flow assertion updated

This prevents test breakage when the text change deploys.


Positive Notes

  • More concise and action-oriented messaging ("track and manage" vs "360-degree view")
  • Consistent em-dash usage for visual separation
  • Build verified (ReScript + production)

Verdict:Approve — Clean text improvement with proper test synchronization.

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.

2 participants