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 Grid: Storage Add-on Dropdown not displaying for spotlight plan #90905

Closed
jeyip opened this issue May 20, 2024 · 4 comments · Fixed by #91050
Closed

Plans Grid: Storage Add-on Dropdown not displaying for spotlight plan #90905

jeyip opened this issue May 20, 2024 · 4 comments · Fixed by #91050

Comments

@jeyip
Copy link
Contributor

jeyip commented May 20, 2024

Quick summary

There used to be a dropdown that allowed users to purchase storage add-ons for creator / entrepreneur spotlight plans in the plans grid. Something changed and the dropdown is no longer rendered.

Was this intentional? If not, the dropdown should be rendered again.

Screenshot 2024-05-13 at 2 30 44 PM

Steps to reproduce

  1. Ensure no storage add-ons have been applied to the site.
  2. If the site doesn't have a creator or entrepreneur plan, purchase one.
  3. Navigate to /plans
  4. Verify that the spotlight creator / entrepreneur plan doesn't render a storage add-on dropdown, and only shows a storage capacity label of "50GB"

What you expected to happen

There should be a dropdown in the spotlight plan which allow the user to purchase storage add-ons for their current plan.

What actually happened

No dropdown is rendered

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@chriskmnds
Copy link
Contributor

chriskmnds commented May 21, 2024

I've assigned this to myself and looking into it.

Was this intentional? If not, the dropdown should be rendered again.

I think I'm convinced the functionality was deliberately disabled/removed, as we actually don't have the logic anywhere for doing it right. It was probably disabled due to being broken. It needs a bit of tweaking to account for the available storage upgrade.

@chriskmnds chriskmnds added Storage Addon and removed Needs triage Ticket needs to be triaged labels May 21, 2024
@chriskmnds
Copy link
Contributor

Hmmm I've done quite a bit of digging here, and boy is this turning into a challenge! As far as controlling how/what is shown for the current plan, which may involve already purchased upgrades, the way we've defined "add-on" storage slugs resulting to things like FEATURE_50GB_STORAGE_ADD_ON mapping to 100GB strings. Part of me insists on reaping it all out and redoing, which isn't really going to happen.

Things apparently work perfectly well for add-ons page without any form of ambiguity on what is shown/available there. I'm starting from there and working my way out to figure out why/how things needed to diverge for plans-grid.

I'll dive a bit deeper still.

@chriskmnds
Copy link
Contributor

Another issue likely related #90947

Storage just keeps giving. Things look pretty complicated/tangled to me throughout. 😞

@chriskmnds
Copy link
Contributor

chriskmnds commented May 21, 2024

This is also related #90949

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