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

Free Trials: Add Free Trial Hub "Expired" State #1471

Closed
2 tasks done
mikeshelton1503 opened this issue Dec 10, 2015 · 8 comments
Closed
2 tasks done

Free Trials: Add Free Trial Hub "Expired" State #1471

mikeshelton1503 opened this issue Dec 10, 2015 · 8 comments
Assignees
Labels
[Feature] Plans & Upgrades All of the plans on WordPress.com and flow for upgrading plans. [Type] Task

Comments

@mikeshelton1503
Copy link
Contributor

When a free trial expires we are giving the user a 3 day grace period (up to day 17). During that time we need to adjust the Free Trial Hub to indicate that the trial plan has expired and that their features will be removed.

Here is the desktop layout:

image

For reference here's what the active trial state looks like for the Free Trial Hub: #1151 (comment)

cc @danhauk @breezyskies @scruffian @Tug

@mikeshelton1503 mikeshelton1503 added [Feature] Plans & Upgrades All of the plans on WordPress.com and flow for upgrading plans. [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR Free Trials labels Dec 10, 2015
@mikeshelton1503 mikeshelton1503 added this to the No-cc Free Trials: v1 milestone Dec 10, 2015
@breezyskies
Copy link
Contributor

Looks great!

One small copy note: Theme changes will be lost reads to me as "setting changes I made to my current theme will be lost" -- I might change this to Theme will be removed (or similar), assuming they had chosen a premium theme.

@mikeshelton1503
Copy link
Contributor Author

Forgot to include this:

  • The Expired today label in the status box will change to Expired 1 day ago then Expired 2 days ago.
  • The Plan features will be removed in 3 days copy in the status box will change to [...] removed in 2 days then [...] removed in 1 day

@mikeshelton1503
Copy link
Contributor Author

One small copy note: Theme changes will be lost reads to me as "setting changes I made to my current theme will be lost" -- I might change this to Theme will be removed (or similar), assuming they had chosen a premium theme.

My thought was that the most important warning to give the user in this case is "the custom changes will be lost" since you can't get them back. You can always get the theme back (if you pay of course).

However I do agree that also saying the theme will be removed would be helpful, though it's a bit long if we do: Theme and custom changes will be removed

@ghost
Copy link

ghost commented Dec 11, 2015

Theme changes will be lost

Can we show no warning if they are using a free theme and a simple warning if they have a premium theme? If so, I think Theme will be removed that Brie suggested seems cleanest...

@mikeshelton1503
Copy link
Contributor Author

Can we show no warning if they are using a free theme and a simple warning if they have a premium theme? If so, I think Theme will be removed that Brie suggested seems cleanest...

If they're using a free theme then the Unlimited Premium Themes feature should show as Not in use so no, no warning will show in that case. However that doesn't change the concern of wanting to warn users that their custom changes to the the Premium theme will be lost.

@drewblaisdell
Copy link
Contributor

@mikeshelton1503 For v1 of PlanFeatures, is there a way we can tweak the design so that we don't need dynamic features? I'm investigating adding that behavior now, but it would be nice to have something in case we don't implement that in the first version.

@mikeshelton1503
Copy link
Contributor Author

For v1 of PlanFeatures, is there a way we can tweak the design so that we don't need dynamic features?

Yea if we can't do the dynamic features then we could do something like this:

image

It moves things around a bit but should be a good compromise for v1.

cc @breezyskies

@breezyskies
Copy link
Contributor

It moves things around a bit but should be a good compromise for v1.

👍 Looks good.

@drewblaisdell drewblaisdell added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] In Progress [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] In Progress labels Dec 22, 2015
@lancewillett lancewillett removed the [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR label Jan 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Plans & Upgrades All of the plans on WordPress.com and flow for upgrading plans. [Type] Task
Projects
None yet
Development

No branches or pull requests

4 participants