Skip to content

Remove card expiry validation#70

Merged
fz-ryanbigg merged 3 commits intomainfrom
remove-card-expiry-validation
Feb 16, 2026
Merged

Remove card expiry validation#70
fz-ryanbigg merged 3 commits intomainfrom
remove-card-expiry-validation

Conversation

@fz-ryanbigg
Copy link
Contributor

@fz-ryanbigg fz-ryanbigg commented Feb 15, 2026

📝 Description

Jira ticket: DASH-4898

Removing card expiry, as it is not required when:

  1. Updating the alias on a card
  2. Setting the card to be a customer's default card

Change Checklist

The following must be reviewed and signed off by the creator of the PR. To sign off review and check each

🔑 Security

  • Security of any environment is not reduced by this change.
  • All existing security controls remain in place or have been replaced with equal or stronger controls.

Testing of Changes

  • Testing results have been verified as complete and covering all aspects of the change.
  • Evidence of testing attached
  • Security scans are passing
  • No package vulnerabilities introduced

🔎 Quality assurance

  • Reviewed this PR myself
  • Automated tests are required and included
  • Security controls have been reviewed
  • Changes are Blue-Green friendly

Cards can be updated _sans_ expiry, in cases such as:

1. when make_default is passed as an argument
2. When a new alias is being specified
@snyk-io-au
Copy link

snyk-io-au bot commented Feb 15, 2026

Snyk checks have failed. 2 issues have been found so far.

Status Scanner Critical High Medium Low Total (2)
Open Source Security 0 0 2 0 2 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@fz-ryanbigg fz-ryanbigg merged commit af2567d into main Feb 16, 2026
6 of 7 checks passed
@fz-ryanbigg fz-ryanbigg deleted the remove-card-expiry-validation branch February 16, 2026 04:06
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.

2 participants