mozilla-mobile / mozilla-vpn-client

A fast, secure and easy to use VPN. Built by the makers of Firefox.
https://vpn.mozilla.org
Other
454 stars 110 forks source link

Buttons in "Subscription Expiring" message do not work #6757

Open data-sync-user opened 1 year ago

data-sync-user commented 1 year ago

Mozilla VPN affected versions:

Platform tested:

Prerequisites:

Steps to reproduce:

  1. Go to the messages screen
  2. Click the “Your subscription is ending soon” message
  3. Try clicking either of the CTA’s

Expected results:

Actual results:

┆Issue is synchronized with this Jira Bug ┆Reporter: Matt Lichtenstein

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Santiago Andrigo Where are these buttons supposed to link to? It looks like when they were originally implemented, they were both opening to to web urls

Manage accounts was going to: https://accounts.firefox.com/signin ( https://accounts.firefox.com/signin?email=a@b.c )

Contact support was linking… nowhere (Beatriz Rizental Machado maybe I am wrong here?) - should this be linking to one of the web forms (http://subscriptions.firefox.com ( http://subscriptions.firefox.com/ ) or https://support.mozilla.org/en-US/products/firefox-private-network-vpn+ ( https://support.mozilla.org/en-US/products/firefox-private-network-vpn+|smart-link ) - which both have different support forms), or to the in-app support page?

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

Matt Lichtenstein

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Santiago Andrigo As far as I can tell (Andrea Marchesini can correct me if I am wrong) we don’t currently have a way to deep link to the contact support form, so there would need to be some changes to the client. Possibly significant changes that will likely delay the RC.

Do we want to:

a) implement these changes so we can deep link

b) deep link to the “get help” page instead (this is already supported, but we’d probably want to change the CTA string in the message from “contact support” to “get help”… is it too late for this due to translations?)

c) link out to one of the web support forms mentioned in my previous comment

d) disable this message for 2.15, and do option a) or b) for 2.16

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

This is not important enough to allocate further effort. Seems like (c) is a good path forward.

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Santiago Andrigo regarding this

{quote}For the first CTA: Let’s send them to https://subscriptions.firefox.com/subscriptions ( https://subscriptions.firefox.com/subscriptions ) please.{quote}

It turns out we already have the logic to send them to either FxA, apple subscriptions, or google subscriptions based on there subscription type, so unless there are any objections, I will implement it so that the CTA takes them to where the subscription was purchased from

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

Ah, that’s great! I had completely forgotten about the mobile subs, thanks for catching that.

data-sync-user commented 1 year ago

➤ Valentina Virlics commented:

Verified this on across platforms on the last 2.15 builds.

The following are happening:

Matt Lichtenstein Please let us know if we should track the crash separately.

[^mozillavpn (3).txt]

If the crash is tracked here, we should reopen this ticket.

data-sync-user commented 1 year ago

➤ Valentina Virlics commented:

Matt Lichtenstein could you please check out our questions? Thank you!

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Valentina Virlics I’ve summarized the current issues with the subscription expiring message here ( https://mozilla-hub.atlassian.net/browse/VPN-2542?focusedCommentId=677180 ) - but I have not done any investigation into the issues. Maybe we reopen the original story? what do you think?

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

Note - this code is on 2.15 and should be QA’d, but it’s for a feature that is disabled on 2.15. Hence, this is not a 2.15 blocker.

data-sync-user commented 1 year ago

➤ Valentina Virlics commented:

Santiago Andrigo We cannot verify this on 2.15, as the feature is disabled. Should we remove the fixVersion or what would be the proper approach for this ticket?

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

Switching to Blocked.

Matt Lichtenstein Lesley Norton As per comments in this: https://mozilla-hub.atlassian.net/browse/VPN-2542 ( https://mozilla-hub.atlassian.net/browse/VPN-2542|smart-link ) , should we re-enable this in main so that QA can test if this works in 2.17?

If so, can we create a ticket for this re-enablement and relate it to this ticket as ‘Blocking’?

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Santiago Andrigo I think we should re-enable it and test it whenever we prioritize releasing it, meaning all the other bugs that come with it are prioritized at the same time. Otherwise we are going to re-enable it just to disable it again if it doesn’t go out by the time we cut the next RC’s

data-sync-user commented 1 year ago

➤ Lesley Norton commented:

Santiago Andrigo Done with caveat: I created https://mozilla-hub.atlassian.net/browse/VPN-5006 ( https://mozilla-hub.atlassian.net/browse/VPN-5006|smart-link ) to enable the message, but on stage for now only.

data-sync-user commented 1 year ago

➤ Santiago Andrigo commented:

Matt Lichtenstein Agreed, but afaik all issues have been resolved or we have decided to go forward without them:

data-sync-user commented 1 year ago

➤ Matt Lichtenstein commented:

Santiago Andrigo - in that case, do you want to enable it for 2.16?

data-sync-user commented 1 year ago

➤ Valentina Virlics commented:

Santiago Andrigo for <“Contact support” button crashes the app on iPad Air 2 ---> We should have a ticket for this. If it’s specific to the iPad Air 2, then I think it’s not a blocker either.“> - we will log a tracking ticket once the feature is re-enabled and I can still see it happening. Thanks!