Auditing: storage add-on UI issues

[southp]

1. The same add-on cannot be repurchased from /add-ons after cancelling.

CleanShot.2023-12-26.at.16.47.31.mp4

1. Purchase a storage add-on from /add-ons
2. Cancel it from the purchase management page
3. Go to /add-ons again. The button will stay as "Manage add-on", while it should be "Buy add-on".

From a quick glance, using transaction history to determine whether one has owned an active storage add-on is wrong. This issue will also cause other anomalies in /plans, but I didn't include them here since the root cause is the same. This might potentially relate to Automattic/martech#2418.

2. A user can't buy the 50G add-on and then buy the 100G add-on

As shown here, my test Business site owning a 50G add-on can't buy the 100G add-on since it's absent. In /plans, it's even more confusing since I can select it but it will show me "Contact Support"

It should be available since the maximum storage is 200G. From a quick glance of the code, the way exceedsSiteStorageLimits computed is wrong.

3. The confusing "Contact Support" button when owning a storage add-on.

When owning a storage add-on, picking a lower-tier storage in /plans will show "Contact support":

it also applies to a higher-tier plan:

This relates to Automattic/martech#1458. However, even with the Downgrade button, it won't be less confusing. My recommendation is to simply hide the lower-tier storage options in /plans, which is not the best place to downgrade anyway.

4. Inconsistent price combining of the spotlight plan card

CleanShot.2023-12-26.at.18.17.14.mp4

As demonstrated above, when we first select a storage add-on before owning one, it shows the combined price in the spotlight plan card. After that, it turns into just the plan price with some flickering between texts.

I'm not sure what's the best behavior here. It's confusing to me to show the combined price, since there is a "+X / month" under the dropdown. I'd prefer us to show them separately, sort of like the final state in the screencast. However, that doesn't align with how the other places in /plans work.

@vinimotaa Could you share your insights here? I'd be grateful.

5. Runtime error by typing certain keystrokes

See #82546

6. The drop-down state is shared between sites.

CleanShot.2023-12-26.at.16.52.41.mp4

As demonstrated above, the drop-down state stays after changing sites. This might be fine, but I'm still documenting it here in case it actually causes other issues.

[JanaganSaravanan]

Hi! I would like to work on this issue. Can you assign this ? And give me some head start if possible. Thank you,

[maciejpilarski]

The issue I worked on - 1. The same add-on cannot be repurchased from /add-ons after cancelling.

📌 REPRODUCTION RESULTS
– Tested on Atomic – Replicated

📌 FINDINGS/SCREENSHOTS/VIDEO
I was able to create the issue on my test site premiumplantest2.wordpress.com, after the canceling the 50GB upgrade, I am unable to repurchase it, I can only buy the other option that was never purchased 100GB

📌 ACTIONS
– Triaged

📌 Message to Author
Hi @southp!

Thanks for your detailed reports. I appreciate your effort, but would also ask to file all the issues as separate bugs, not a combo bug with 6 issues listed. This way it's easier for us and the dev to triage and work on the issues.

I have worked on your first issue and was able to reproduce it. Are you able to modify your report and file the remaining five reports separately?

Thanks in advance!

[southp]

@maciejpilarski Thanks for the suggestion. You are right that I should have broken it down into separate issues from the first place. It's now broken down accordingly:

• Storage add-on: the same add-on cannot be repurchased from /add-ons after cancelling. #86147
• Storage add-on: A user can't buy the 50G add-on and then buy the 100G add-on #86150
• Storage add-on: The confusing "Contact Support" button when owning a storage add-on. #86151
• Storage add-on: Inconsistent price combining of the spotlight plan card #86154
• Storage add-on: The drop-down state is shared between sites.  #86152
• Storage Upgrade Add-on: Dropdown runtime error #82546

Since you mentioned that you are working on the 1st issue, I've also assigned that to you :)

[southp]

Hey @JanaganSaravanan , thanks for pitching in.

Hi! I would like to work on this issue. Can you assign this ? And give me some head start if possible. Thank you,

Now that it has been broken down granular, you can feel free to grab one that interests you the most. We can then move on our conversation there, including the head start pointers. Does that work for you?

[JanaganSaravanan]

Yes! Absolutely. Thank you,

[dzver]

Adding one thing here

The dropdown is only available on Annual plans. It should be available for 2 and 3-year plans, assuming the monthly option doesn't make sense.

CC @southp for confirmation

[southp]

Good quesiton, @dzver . I raised the same question when this project first started, but the decision was that the storage add-on would only have the annual term in this phase. Back then it was not obvious, but it's now revealed by the term dropdown.

@gmovr Do we have 2-year and 3-year products for the storage add-ons?

If not, I think it'd still be better to enable the storage drop-down for 2-year & 3-year plans, which will add the annual storage add-on subscription to their cart. We'd just need to be cautious on how it is communicated on the UI. Does that make sense?

[dzver]

We don't have 2 and 3-year storage add-ons and we don't plan adding such. Despite that, the upgrade can still be purchased from the add-ons page.

It looks like a bug right now but not a high-priority one.

[southp]

Cool. I've created a tracking issue here: Automattic/martech#2614.

[ddc22]

It looks like a bug right now but not a high-priority one.

@dzver
I think this was deliberately hidden due to the lack of calrity in the requirement at time of development (CC - @jeyip) Just want to ping all of you unless there is some other reason we should NOT be doing this (Show add ons in all multi year plans) :)

wp-calypso/client/my-sites/plans-grid/lib/is-storage-upgradeable-for-plan.ts

Lines 3 to 10 in 580f6da

	/**
	* Don't show storage add-on-upsells for:
	* - the enterprise plan which has no storage options
	* - plans that only have 1 default storage option ( and no upgrades )
	* - monthly or multi-year plans
	* - environments with a disabled feature flag
	*/
	export const isStorageUpgradeableForPlan = ( {

Working on this @ #86807

[ddc22]

@dzver
I did some exploration and had some feedback from @jeyip . First I want to clarify and understand what exactly you're suggestion is... I can guess two approaches that might be on your mind.

Sell the annual renewing storage add on with multi year renewing plan

If you meant this approach, then we have some UI complexity involved on the plans grid. Because the Primary price displays a "monthly" price as well as a "Relevant term renewing price". For example if its a 3 year plan we show a calculated monthly price as well as the price to be paid at the end of 3 years.

We will have to do some frontend calculations to come up with the equivalent monthly and relevant term renewing price for storage add-on. But even then there is a chance for confusion, since even though we say that it costs a given amount at the end of a given term, the storage add on will always require payment annually. This is misleading given that the drop down says "Pay every 3 years".

Checkout however shows a better view of this context which I am not sure if we can replicate in the plans grid.

---

Setup relevant multi year renewing storage add ons to match the multi year renewing plan

This would make things much more simpler. But I am not sure of the possiblity of doing this since this was not attempted in the first place.

---

PR available to explore and play around with this options enabled
#86807

The calculations are currently incorrect, but the checkout works well.

CC - @gmovr @vinimotaa @saygunnyc @isatuncman-auto @jeyip

[dzver]

If the calculations are difficult to implement, I'd rather abandon the task than complete it. This is only worth fixing if it's easy.

[ddc22]

Abandoning PR for now due to complexity and low priority of task.

[jeyip]

6. The drop-down state is shared between sites.

Addressing this low-hanging fruit in #87342