Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Recalculate payment total when order total decreases #142

Merged
merged 2 commits into from
Jan 2, 2018

Conversation

dholdren
Copy link
Contributor

@dholdren dholdren commented Dec 4, 2017

  • Apple Pay does not send street address until the last step. If street
    is a factor in the tax calculation you will have a discrepency after
    the payment is made. The payment amount wasn't decreased when
    the order total was lower

- Apple Pay does not send street address until the last step. If street
  is a factor in the tax calculation you will have a discrepency after
  the payment is made. The payment amount wasn't decreased when
  the order total was lower
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks. I would like to see a test for this.

@dholdren
Copy link
Contributor Author

dholdren commented Dec 8, 2017

I added a spec

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

thanks

@tvdeyen
Copy link
Member

tvdeyen commented Dec 11, 2017

@dholdren could you please address the Rubocop offenses? Unfortunately it's part of our Travis builds and thats why its failing.

NOTE TO SELF: Use HoundCI instead of Travis CI for Rubocop.

@dholdren
Copy link
Contributor Author

@tvdeyen sorry for the long delay, the rubocop offenses have been addressed

@tvdeyen tvdeyen merged commit 79d366e into solidusio:master Jan 2, 2018
@tvdeyen
Copy link
Member

tvdeyen commented Jan 2, 2018

Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants