-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Purchase Management: Free purchases shouldn't show "excludes tax" #89121
Comments
PR for this in #89194. :) |
Re-opening because, as David pointed out (#89194 (review)), the fix above targets only purchases where |
I have been working as a React developer for 6 years and recently started exploring Calypso. Over the past month, I have been familiarizing myself with the project documentation and codebase. I now feel ready to contribute and would like to start with an easier task. Do you think this task would be a suitable first contribution to the project? Can I assign it to me to fix it and include all free purchases? |
@oalagial You're welcome to give this a try! |
@sirbrillig To reproduce the issue, I would need to purchase a plan. Is there a way to simulate this process using a dummy user or by utilizing my existing account? Additionally, it would be helpful if you could guide me on how to direct similar questions to a specific member of the project privately. Thank you for your assistance! |
Yes, @oalagial, we have a process for making purchases in a non-live environment we call the "store sandbox". If you could provide me your wordpress.com username I can set you up. |
Hello, |
Thanks. I've emailed you instructions for making test purchases at the email address registered with your account. |
Should be fixed now by #97937 |
Quick summary
For a free purchase (for example a domain connection subscription that is free with a plan) we are showing "excludes tax" in the Renewal Price section, which doesn't make sense:
Steps to reproduce
What you expected to happen
It should have just said "Renewal Price: Free with Plan"
What actually happened
It also said "excludes tax"
Impact
Some (< 50%)
Available workarounds?
Yes, easy to implement
Platform (Simple and/or Atomic)
No response
Logs or notes
No response
The text was updated successfully, but these errors were encountered: