Automattic / newspack-plugin

An advanced open-source publishing and revenue-generating platform for news organizations.
https://newspack.com
GNU General Public License v2.0
324 stars 48 forks source link

fix: display transaction fee as an amount instead of a percentage #3206

Closed dkoo closed 2 weeks ago

dkoo commented 3 weeks ago

All Submissions:

Changes proposed in this Pull Request:

Displays the transaction fee as an amount instead of a percentage in the Donate modal checkout flow.

How to test the changes in this Pull Request:

  1. Check out this branch
  2. While using the Newspack Reader Revenue platform, in Newspack > Reader Revenue > Stripe Settings, enable the "Allow donors to cover transaction fees" option but leave the custom message field blank.
  3. As a reader, start a donation transaction and proceed to the payment part of the modal.
  4. Confirm that the default message shows the fee in an absolute currency amount instead of as a percentage:
Screenshot 2024-06-27 at 4 21 55 PM

Other information:


dkoo commented 2 weeks ago

Note: this reverts changes from https://github.com/Automattic/newspack-plugin/pull/2746

Yes, but also simplifies the display further by not showing the percentage at all, which is hopefully more user-friendly. Thanks for the review!

matticbot commented 1 week ago

:tada: This PR is included in version 4.5.0-alpha.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 1 week ago

:tada: This PR is included in version 4.5.0-hotfix-add-attributes-loader-effect-hook-deps.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 1 week ago

:tada: This PR is included in version 4.6.0-alpha.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 3 days ago

:tada: This PR is included in version 4.6.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: