Skip to content

Fix bug where payment method wouldn't refresh until after page load#1683

Merged
Myztiq merged 3 commits intomasterfrom
SAN-4839-refresh-cc-information
Aug 24, 2016
Merged

Fix bug where payment method wouldn't refresh until after page load#1683
Myztiq merged 3 commits intomasterfrom
SAN-4839-refresh-cc-information

Conversation

@Myztiq
Copy link
Copy Markdown
Contributor

@Myztiq Myztiq commented Aug 24, 2016

We now update the whitelists model when the payment method is added so we know it's been refreshed in the UI.

@runnabot-brother
Copy link
Copy Markdown

The latest push to PR-1683 has failed to build. Check out the logs SAN-4839-refresh-cc-information-runnable-angular

@runnabot
Copy link
Copy Markdown

The latest push to PR-1683 is running on SAN-4839-refresh-cc-information-runnable-angular

@runnabot
Copy link
Copy Markdown

runnabot commented Aug 24, 2016

The latest push to PR-1683 is running on SAN-4839-refresh-cc-information-angular-unit

@Nathan219
Copy link
Copy Markdown
Member

Tested with local and gamma, looks good! 👍

.then(function (res) {
return savePaymentMethod(res.id);
})
.then(function () {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

how does fetching whitelist affect payment method?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

When they put in a payment method the whitelist object needs to be refreshed because the whitelist object contains a boolean flag "hasPaymentMethod"

@Myztiq Myztiq merged commit 80e33f1 into master Aug 24, 2016
@Myztiq Myztiq deleted the SAN-4839-refresh-cc-information branch August 24, 2016 21:30
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

Successfully merging this pull request may close these issues.

5 participants