Skip to content
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

fix: replace Card Name with Cardholder Name #3714

Closed
9 of 11 tasks
Benunc opened this issue Sep 28, 2018 · 2 comments
Closed
9 of 11 tasks

fix: replace Card Name with Cardholder Name #3714

Benunc opened this issue Sep 28, 2018 · 2 comments
Assignees

Comments

@Benunc
Copy link
Member

Benunc commented Sep 28, 2018

Bug Report

User Story

As a Donor, I frequently confuse "Card name" to mean "American Express/Visa/Mastercard/etc" when it should read "Cardholder Name" for simplicity and clarity.

Current Behavior

Currently in Give core on this line https://github.com/WordImpress/Give/blob/master/includes/forms/template.php#L981

...and give-stripe in this line
https://github.com/WordImpress/Give-Stripe/blob/master/includes/give-stripe-helpers.php#L157

...and twocheckout on this line
https://github.com/WordImpress/Give-2Checkout/blob/master/includes/give-twocheckout-helpers.php#L85

It reads "Card name" and even the associated tooltips are not very helpful "The name printed on the front of the card" could also be "American Express"

Expected Behavior

I expect the Field to read "Cardholder Name" (note capitalization) and all associated tool tips to be more helpful as well. Something like "The name of the credit card account holder."

Bug Type

  • This bug describes functionality that once worked as expected in version X.X.X.
  • This bug describes functionality that never worked as expected.
  • I am not sure whether this functionality ever worked as expected.

Steps to Reproduce

  1. Enable any add-on that accepts cards
  2. The field is displayed confusingly.

Acceptance Criteria

  • The field label is changed from Name on the Card to Cardholder Name.
  • The input placeholder is changed from Card Name to Cardholder Name.
  • The tooltip reads: The name of the credit card account holder.
  • The above changes have been verified in the following plugins:
    • Give core
    • Stripe
    • Square
    • 2Checkout
@kevinwhoffman kevinwhoffman changed the title fix: clarify Card name to mean cardholder name fix: replace Card Name with Cardholder Name Oct 3, 2018
@kevinwhoffman
Copy link
Contributor

@nishitlangaliya See the acceptance criteria above. You should submit a PR for Give core and a PR for 2Checkout to update the strings in question.

Mehul has already updates Stripe so no need to worry about that.

@mehul0810
Copy link
Contributor

mehul0810 commented Oct 3, 2018

@kevinwhoffman Just to update, I've updated this issue for Square, not Stripe. I'll update it for Stripe today as I'm doing all the tasks of Stripe. I've updated the description above as well. Let me know if there is any change.

@ravinderk ravinderk reopened this Oct 3, 2018
ravinderk added a commit that referenced this issue Oct 3, 2018
fix: replace `Card Name` with `Cardholder Name` #3714
@ravinderk ravinderk reopened this Oct 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants