Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Disables funding with debit/credit card. Only affects Coinbase #9331

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

bsclifton
Copy link
Member

@bsclifton bsclifton commented Jun 8, 2017

Test plan

  1. Be on an IP that is geo-located in the USA (via VPN or physically being in USA)
  2. Go to Preferences > Payments
  3. Click "Add funds"
  4. "Fund with debit/credit" button should be disabled; appropriate messaging should be displayed

screen shot 2017-06-07 at 11 14 40 pm

Description

Disables funding with debit/credit card. Only affects Coinbase (ex: customers in USA)

Fixes #9327

Auditors: @cezaraugusto, @NejcZdovc

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).
  • Tagged reviewers and labelled the pull request as needed.

Reviewer Checklist:

Tests

  • Adequate test coverage exists to prevent regressions
  • Tests should be independent and work correctly when run individually or as a suite ref
  • New files have MPL2 license header

@bsclifton
Copy link
Member Author

Please note: this change was done in a way that we can easily enable/disable the UI in case of a service outage (UI looks at a single boolean for enabled / disabled)

…ustomers in USA)

Fixes #9327

Auditors: @cezaraugusto, @NejcZdovc

Test Plan:
1. Be on an IP that is geo-located in the USA (via VPN or physically being in USA)
2. Go to Preferences > Payments
3. Click "Add funds"
4. "Fund with debit/credit" button should be disabled; appropriate messaging should be displayed (see issue)
@bsclifton bsclifton force-pushed the disable-fund-with-cc branch from 2adba93 to 2a2aaa1 Compare June 8, 2017 06:57
@bsclifton bsclifton requested a review from luixxiul June 8, 2017 06:58
Copy link
Contributor

@luixxiul luixxiul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@luixxiul
Copy link
Contributor

luixxiul commented Jun 8, 2017

I have some ideas to polish the button. I'll give a try with another PR.

@bbondy
Copy link
Member

bbondy commented Jun 8, 2017

Thanks for taking this clifton, merging.

@bbondy bbondy merged commit 7c7504e into master Jun 8, 2017
bbondy added a commit that referenced this pull request Jun 8, 2017
Disables funding with debit/credit card. Only affects Coinbase
bbondy added a commit that referenced this pull request Jun 8, 2017
Disables funding with debit/credit card. Only affects Coinbase
Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

@bsclifton bsclifton deleted the disable-fund-with-cc branch June 8, 2017 16:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants