Skip to content

Payment Methods: use full available width (#2360) - #2364

Merged
xegea merged 2 commits into
mainfrom
claude/issue-2360
Nov 5, 2026
Merged

xegea merged 2 commits into
mainfrom
claude/issue-2360

Conversation

@xegea

@xegea xegea commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Closes #2360

Removes the maxWidth: 1000 constraint on the Base Payment Methods page container so the list, search and status filter use the full content width. Layout-only; no API, migration or test change (no logic involved).

Decisions taken: "Payment Methods page" is read as Cordel → Base Payment Methods, the only Payment Methods catalogue page that had a max-width.


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UJjrZJXHBDyw9pH79CnZmf

xegea commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

Parked: not mergeable until main is fixed.

API shards 1 and 3 fail on plan-snapshot-dual-write.test.ts ("committing makes the snapshot active…"). It re-runs migration 347's backfill, which selects source_plan_snapshot_id, a column migration 356 (#2125 stage 17) dropped. main (17bd77a) carries both, so it should fail the same way. This PR's own checks (Admin, Payment page, CodeQL) are green. Once a person updates or retires those tests on main, this PR needs a merge of main and a CI re-run.


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@xegea
xegea merged commit abaac30 into main Nov 5, 2026
4 of 6 checks passed

This branch was previously deployed

1 inactive deployment
dev — 967eaf40 Deployed Nov 5, 2026 by xegea via Admin #3543
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Full Available Window Width for Payment Methods

2 participants