New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BANK-713] Make CVV non-null for all cases of PUT /cards/{id} #330

Merged

ArneCircle merged 2 commits into circlefin:master from Wongmat:BANK-713/add-fields-2

Jul 11, 2023

Contributor

Wongmat commented Jul 11, 2023 •

edited

Loading

Related: [BANK-713] Add billing detail fields to PUT /cards/{id} #328
This PR makes some slight adjustments after speaking with risk and deciding that we should require CVV for all updates.
This follows https://github.com/circlefin/platform-eft/pull/5269

Wongmat added 2 commits

July 11, 2023 19:47


          [BANK-713] Make CVV non-null for all cases of PUT /cards/{id}

ee22389


          Trigger Build

511cc78

Signed-off-by: Mat Wong <mat.wong@circle.com>

ArneCircle approved these changes

View reviewed changes

ArneCircle merged commit f0db6c9 into circlefin:master

Wongmat deleted the BANK-713/add-fields-2 branch

July 11, 2023 22:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet