Skip to content

fix: fix expiry date input in dc#264

Merged
jakubjasinsky merged 1 commit into
masterfrom
fix-expiry-date-input
Jun 3, 2026
Merged

fix: fix expiry date input in dc#264
jakubjasinsky merged 1 commit into
masterfrom
fix-expiry-date-input

Conversation

@jakubjasinsky
Copy link
Copy Markdown
Collaborator

Summary

Fix expiry date validation error not displaying in Dynamic Checkout. The
card.invalid-date error code wasn't handled, so a fully invalid expiry date
silently failed to surface its error message.

Changes

  • Handle card.invalid-date alongside card.invalid-month /
    card.invalid-year in the card field error handler, so the expiry-date error
    message is shown.

card.invalid-date falls through the same case block as the existing month/year
errors, so it now targets the expiry-date-error-message element. Minimal,
single-line change in src/dynamic-checkout/payment-methods/card.ts.

Copilot AI review requested due to automatic review settings May 28, 2026 14:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes missing handling of the card.invalid-date error code in Dynamic Checkout's card validation, ensuring the expiry-date error message is surfaced when a fully invalid expiry date is entered.

Changes:

  • Add card.invalid-date as a fall-through case alongside card.invalid-month / card.invalid-year in CardPaymentMethod.handleValidationError, targeting the existing expiry-date-error-message element.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jakubjasinsky jakubjasinsky merged commit d7024ae into master Jun 3, 2026
5 checks passed
@jakubjasinsky jakubjasinsky deleted the fix-expiry-date-input branch June 3, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants