-
Notifications
You must be signed in to change notification settings - Fork 90
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.
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
Amex CVV on front #6
Comments
For that we must change layout little bit. I don't know if it's worth it |
I think it is worth it. Came here to open this same issue. People get confused easily and this can drop AMEX conversions significantly. |
I think it would be nice if the layout is customizable because in my own case I have been looking for how to show only the CardHoler Name at the front while the Card Number, Expiry & CVV will be at the back. Worth looking into. Great Job BTW! |
+1 on this, definetly worth it. The main reason for having a visual credit card on the page is not because it looks cool (although it does) but to help people understand what information is being asked from them. Amex cards have numbers both in the front and back, and they need to input the one in the front, currently they are shown that they need to input the one in the back! |
Any news on merging this amex fix pull req? |
Very very nice work! One small thing: for American Express, the CVV number is printed on the front unlike most other cards. Would be nice to allow for this.
The text was updated successfully, but these errors were encountered: