Skip to content
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

Closed
DavidRothstein opened this issue Apr 2, 2024 · 9 comments · Fixed by #89194
Closed

Purchase Management: Free purchases shouldn't show "excludes tax" #89121

DavidRothstein opened this issue Apr 2, 2024 · 9 comments · Fixed by #89194
Labels
[Pri] Low Address when resources are available. Quick Fix Triaged To be used when issues have been triaged. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@DavidRothstein
Copy link
Contributor

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:

free-domain-connection

Steps to reproduce

  1. Buy a plan
  2. Add a free domain connection
  3. Go to https://wordpress.com/me/purchases and click through to manage the domain connection purchase

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

@DavidRothstein DavidRothstein added [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged Quick Fix labels Apr 2, 2024
@github-actions github-actions bot added the [Pri] Low Address when resources are available. label Apr 2, 2024
@Aurorum
Copy link
Contributor

Aurorum commented Apr 4, 2024

PR for this in #89194. :)

@cat-og cat-og moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Apr 5, 2024
@cat-og cat-og added Triaged To be used when issues have been triaged. and removed Needs triage Ticket needs to be triaged labels Apr 5, 2024
@sirbrillig
Copy link
Member

Re-opening because, as David pointed out (#89194 (review)), the fix above targets only purchases where isIncludedWithPlan() is true, not all free purchases.

@sirbrillig sirbrillig reopened this Apr 10, 2024
@oalagial
Copy link
Contributor

oalagial commented Dec 27, 2024

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?

@sirbrillig
Copy link
Member

@oalagial You're welcome to give this a try!

@oalagial
Copy link
Contributor

oalagial commented Dec 30, 2024

@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!

@sirbrillig
Copy link
Member

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.

@oalagial
Copy link
Contributor

oalagial commented Jan 1, 2025

Hello,
Thank you very much. My username in wordpress is orestisal97

@sirbrillig
Copy link
Member

sirbrillig commented Jan 2, 2025

Thanks. I've emailed you instructions for making test purchases at the email address registered with your account.

@sirbrillig
Copy link
Member

Should be fixed now by #97937

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] Low Address when resources are available. Quick Fix Triaged To be used when issues have been triaged. [Type] Bug When a feature is broken and / or not performing as intended
Development

Successfully merging a pull request may close this issue.

5 participants