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

Premium theme not activated when signing up with Premium plan or above #21075

Closed
seear opened this issue Dec 22, 2017 · 9 comments
Closed

Premium theme not activated when signing up with Premium plan or above #21075

seear opened this issue Dec 22, 2017 · 9 comments
Labels
[Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Signup & Account Creation All screens and flows for making a new WordPress.com account. [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. NUX [Pod] Design Selection Issues related to the WP.com Design Selection focus pod. [Pri] Normal Schedule for the next available opportuinity. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@seear
Copy link
Contributor

seear commented Dec 22, 2017

Steps to reproduce

  1. Starting at URL: (logged out of WordPress.com): https://wordpress.com/theme/photo-blog (or any other premium theme)
  2. Select "Pick this design"
  3. Complete the signup flow with at least a Premium plan

What I expected

Site should have the photo-blog theme activated after checking out.

What happened instead

Site has a default theme activated.

Browser / OS version

n/a

Screenshot / Video

Context / Source

Manual testing while investigating #20717

Notes

This happens because the Premium plan includes themes, so the theme is not added to the cart, but headstart does not allow switching to a premium theme (see class-headstart-lifecycle#maybe_switch_to_theme in Headstart repo).

@seear seear added NUX [Feature] Signup & Account Creation All screens and flows for making a new WordPress.com account. [Feature Group] Appearance & Themes Features related to the appearance of sites. [Type] Bug When a feature is broken and / or not performing as intended labels Dec 22, 2017
@apeatling apeatling modified the milestones: Product Quality, Product Quality: Site Setup Flows Jan 31, 2018
@apeatling apeatling added the [Pri] High Address as soon as possible after BLOCKER issues label Feb 1, 2018
@lancewillett
Copy link
Contributor

I'd like to also fix this bad one: #14896

We shouldn't be charging customers for themes after they've upgraded.

@rachelmcr
Copy link
Contributor

I tested and confirmed that this is still happening.

If I go to My Sites > Themes I can then activate the premium theme on my site at no additional charge, but if I go back to the URL I started at (https://wordpress.com/theme/photo-blog) I run into #14896 which is really bad.

@donpark
Copy link
Contributor

donpark commented Jul 16, 2019

Just started digging into this one.

One of the problems is that ThemeSheet, class that renders details about a theme, sort of works but can behave weirdly when site has not been specified. For example, it display "Pick this design $36" and, when its clicked, pops open ThemesSiteSelectorModal with a site pre-selected.

What action this flow takes is depends mostly on which site is selected (see themes-options.jsx) but it's only determined once at some point before ThemesSiteSelectorModal is opened and is not updated when site selection changes. If it did, wordings on the modal dialog would change from "Purchase on:" to "Activate on:" or "Customize on:" and main action button will proceed to non-purchase flow. I suspect ThemeSheet part of Calypso was originally designed to work with a site already created and specified then retrofitted later to be used during signup as well as through 'all-sites' mode.

But the fix appears to be a lot more complicated than just adding onSiteChanged callback to SiteSelectorModal and use the callback to dynamically determine what to show and where to flow based on which site is currently selected because what needs to be set is injected (Redux connect) from outside the modal.

@donpark
Copy link
Contributor

donpark commented Jul 17, 2019

Some ideas of best way to proceed:

  1. To address The Premium plan should not need to buy the premium theme #14896, change how we handle /theme to work like how /customize is handled: require site selection before showing themes. This will fix the problem of Premium Plan site owners having to pay for premium themes and avoid the need to refactoring ThemeSheet to support dynamically changing target site.

  2. To address this issue of how to handle theme selection before site plan is chosen, I think we can either patch headstart to activate premium themes or, if that is blocked by some unknown conditions, schedule an auto-theme switch as last step in the headstart.

@donpark
Copy link
Contributor

donpark commented Aug 14, 2019

@seear "Pick this design" button currently does not allow me to create a new website a selected Premium theme. Is this issue still valid?

UPDATE: Only just now realized I should try the test flow without being signed in. Doh.
UPDATE2: Yup, not being signed in leads to signup but now having difficulty running headstart in the sandbox. I'll need to research how all the sandbox related defines work and find the right mix to test with.

@lancewillett lancewillett removed the [Pri] High Address as soon as possible after BLOCKER issues label Aug 28, 2019
@github-actions
Copy link

This issue is stale because it has been 180 days with no activity. You can keep the issue open by adding a comment. If you do, please provide additional context and explain why you’d like it to remain open. You can also close the issue yourself — if you do, please add a brief explanation and apply one of relevant issue close labels.

@obenland obenland added the [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. label Mar 17, 2021
@ianstewart
Copy link
Contributor

Adding this to the Design Selection program we're looking at for issue triage. Sounds fixed-ish? But we should test it. :)

@ianstewart ianstewart added the [Pod] Design Selection Issues related to the WP.com Design Selection focus pod. label May 4, 2021
@kwight kwight added the [Pri] Normal Schedule for the next available opportuinity. label May 6, 2021
@andres-blanco andres-blanco self-assigned this Jun 2, 2021
@andres-blanco
Copy link
Contributor

I confirm that this bug exists and it's reproducible.

A different theme needs to be used since the one cited above (photo-blog) does not exist anymore. You can use this: https://wordpress.com/theme/ovation

I am searching for possible fixes, seems complex

@andres-blanco andres-blanco removed their assignment Jun 9, 2021
@ianstewart
Copy link
Contributor

I am searching for possible fixes, seems complex

With the complexity and our priorities I'm going to close this one as wontfix. We can open it up again if there are user reports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Signup & Account Creation All screens and flows for making a new WordPress.com account. [Feature] Theme Showcase The theme showcase screen in Calypso in Appearance > Themes. NUX [Pod] Design Selection Issues related to the WP.com Design Selection focus pod. [Pri] Normal Schedule for the next available opportuinity. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

9 participants