Skip to content

Remove card expiry validation#70

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

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

Conversation

@fz-ryanbigg

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

Copy link
Copy Markdown
Contributor

📝 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

snyk-io-au Bot commented Feb 15, 2026

Copy link
Copy Markdown

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