Skip to content

fix: cancelled with double 'l'#6973

Merged
elena-zh merged 4 commits intodevelopfrom
fix-to-cancelled
Feb 5, 2026
Merged

fix: cancelled with double 'l'#6973
elena-zh merged 4 commits intodevelopfrom
fix-to-cancelled

Conversation

@elena-zh
Copy link
Contributor

@elena-zh elena-zh commented Feb 5, 2026

PR fixes spelling issue for 'cancelled' status: it some places it has 1 'l', in other places it has a double 'l'.

Since in most of cases we use a double 'l', I've added one more 'l' to this word in some places on CoW Swap and Explorer.

Examples:
https://explorer-dev-git-fix-to-cancelled-cowswap-dev.vercel.app/linea/address/0x9fa3c00a92ec5f96b1ad2527ab41b3932efeda58
image

https://explorer-dev-git-fix-to-cancelled-cowswap-dev.vercel.app/orders/0xa11f312ab0363efda7c75bf87fdf747a87b438a5aacd0c854743c749dee0036c9fa3c00a92ec5f96b1ad2527ab41b3932efeda5869733d5a
image

Summary by CodeRabbit

  • Style
    • Standardized user-facing wording to British English: "Canceled" → "Cancelled" and "Canceling" → "Cancelling".
    • Updated step descriptions, tooltips, and table/row labels across the interface to reflect the new spelling.
    • No functional or behavioral changes; purely textual updates visible to end users.

@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cowfi Ready Ready Preview Feb 5, 2026 3:30pm
explorer-dev Ready Ready Preview Feb 5, 2026 3:30pm
swap-dev Ready Ready Preview Feb 5, 2026 3:30pm
widget-configurator Ready Ready Preview Feb 5, 2026 3:30pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
cosmos Ignored Ignored Feb 5, 2026 3:30pm
sdk-tools Ignored Ignored Preview Feb 5, 2026 3:30pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

This PR standardizes spelling from "Canceled" to "Cancelled" across UI strings in cowswap-frontend and explorer: step descriptions, tooltip text, and UI labels. No logic, props, or public signatures were changed.

Changes

Cohort / File(s) Summary
cowswap-frontend spelling corrections
apps/cowswap-frontend/src/modules/ethFlow/pure/EthFlowStepper/index.cosmos.tsx
Updated step description strings to British spelling: "Canceling/Canceled" → "Cancelling/Cancelled". No logic changes.
explorer spelling corrections
apps/explorer/src/components/orders/DetailsTable/detailsTableTooltips.tsx, apps/explorer/src/components/orders/OrdersUserDetailsTable/index.tsx
Replaced "Canceled" with "Cancelled" in tooltip and UI label text and updated related comments. No functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • shoom3301
  • alfetopito

Poem

🐰
I hopped through text both near and far,
Swapped a single letter — now polished, a star.
"Cancelled" I whisper with tidy delight,
Small change, big neatness, all set and right.

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description explains the issue and shows before/after examples with screenshots, but lacks required sections like 'To Test' and 'Background' from the template. Add 'To Test' section with steps to verify the spelling changes are consistent, and consider adding 'Background' context if needed.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: fixing the spelling of 'cancelled' to use double 'l' consistently throughout the codebase.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-to-cancelled

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@elena-zh elena-zh self-assigned this Feb 5, 2026
@elena-zh elena-zh marked this pull request as ready for review February 5, 2026 15:20
@elena-zh elena-zh requested a review from a team February 5, 2026 15:21
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In
`@apps/cowswap-frontend/src/modules/ethFlow/pure/EthFlowStepper/index.cosmos.tsx`:
- Line 228: In EthFlowStepper (index.cosmos.tsx) there are two remaining
American-spelled string literals "'[CANCEL] 1. Open and Canceling'" and
"'[CANCEL+EXPIRED] 1. Expired and Canceling'"; update those string values to use
the British spelling "Cancelling" to match the other entries (e.g., "'[CANCEL]
1. Open and Cancelling'" and "'[CANCEL+EXPIRED] 1. Expired and Cancelling'") so
all occurrences in the component are consistent.

@elena-zh elena-zh merged commit 4a5c09f into develop Feb 5, 2026
14 of 15 checks passed
@elena-zh elena-zh deleted the fix-to-cancelled branch February 5, 2026 15:40
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants