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

Plans: Plan price not updated when storage option selected if a storage add-on previously purchased #90947

Closed
chriskmnds opened this issue May 21, 2024 · 3 comments · Fixed by #91050
Assignees
Labels

Comments

@chriskmnds
Copy link
Contributor

Quick summary

When having purchased a storage add-on (e.g. 50GB) and going to /plans/[site], then selecting any storage option for one of the higher tier plans will not update the price of the respective plan.

Screenshot 2024-05-21 at 5 47 12 PM

Steps to reproduce

  1. Create a site on the Creator plan
  2. Go to /add-ons and purchase a 50GB storage upgrade
  3. Go to /plans/[site] and select any of the options in the dropdown for the Entrepreneur plan
  4. Confirm that the price of the Entrepreneur plan no longer reflects the change

What you expected to happen

Either the storage option NOT being available for selection (if any issues with it) or the price of the Entrepreneur (higher tier) plan to reflect the storage option's price.

What actually happened

The price did not get updated

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@chriskmnds
Copy link
Contributor Author

@southp @jeyip can you please confirm here and/or point to any existing issue if spotted previously?

@jeyip
Copy link
Contributor

jeyip commented May 23, 2024

If any storage add-ons are purchased, user's shouldn't be able to purchase any more storage add-ons ( I'll try and dig up a link for context soon ). I thought I added logic to ensure it wasn't available for selection on the plans page, but it's been quite a while and I can'tdistinctly remember.

The behavior we're seeing now might have been an oversight 🤔

@chriskmnds
Copy link
Contributor Author

If any storage add-ons are purchased, user's shouldn't be able to purchase any more storage add-ons ( I'll try and dig up a link for context soon ). I thought I added logic to ensure it wasn't available for selection on the plans page, but it's been quite a while and I can'tdistinctly remember.

I think that concept and direction (that only a single purchase is permissible) is the root of a lot of issues I'm seeing with the implementation. The design and abstractions would have been a lot more natural if that was never baked into the logic so deeply.

I am also not sure if that is even a restriction on the backend. The error that I get is that I need to purchase storage greater than the previous purchase. I'm investigating that, but I am pretty sure the frontend should have been flexible to reason beyond the confines of either 50 or 100 GB (irrespective of what's made available for selection).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment