Skip to content

feat: revised express checkout copy (MP-3090) - #7270

Merged
tiarascoleman merged 3 commits into
mainfrom
MP-3090
Sep 18, 2026
Merged

tiarascoleman merged 3 commits into
mainfrom
MP-3090

Conversation

@tiarascoleman

Copy link
Copy Markdown
Contributor

Ticket reference: https://kiva.atlassian.net/jira/software/projects/MP/boards/275?filter=&groupBy=none&selectedIssue=MP-3090
Proposed solution: changed copy (the second commit also feature additional cohesive copy edits)
**Proof of testing:**

Screenshot 2026-09-17 at 1 46 05 PM
Screenshot 2026-09-17 at 1 46 20 PM

@tiarascoleman
tiarascoleman requested a review from a team September 17, 2026 17:47
@tiarascoleman tiarascoleman self-assigned this Sep 17, 2026
@tiarascoleman tiarascoleman added the b2c Sends B2C team a message in Slack on PR creation label Sep 17, 2026
@tiarascoleman tiarascoleman changed the title feat: revised express checkout copy feat: revised express checkout copy (MP-3090) Sep 17, 2026
Comment thread src/components/Thanks/ExpressCheckout/ExpressCheckoutModal.vue Outdated
Comment thread src/components/Thanks/ExpressCheckout/ExpressCheckoutModal.vue Outdated
@cristhianDt
cristhianDt self-requested a review September 17, 2026 17:54

@dyersituations dyersituations left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Mainly see comment about loading state vs loaded state.

@cristhianDt cristhianDt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍🏽

@tiarascoleman
tiarascoleman merged commit da6e78c into main Sep 18, 2026
5 checks passed
@tiarascoleman
tiarascoleman deleted the MP-3090 branch September 18, 2026 15:19
const depositRequired = computed(() => (numeral(totalDue.value).value() ?? 0) > 0);
const lightboxTitle = computed(() => (ready.value ? 'Adding to basket' : 'Confirm payment'));

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is the opposite of what the ticket requests, it now says "Adding to basket" when the modal is completely loaded:

Image

const secondaryLabel = computed(() => {
return props.expressCheckoutEnabled
? 'Express checkout'
? 'Add to basket'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The ticket requests to remove this text, not replace it:

Image

@dyersituations

Copy link
Copy Markdown
Collaborator

Ticket requests this button to change to "Add to basket"
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b2c Sends B2C team a message in Slack on PR creation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants