Skip to content

Display the plans modal in the pull an existing site creation flow - #2197

Merged
sejas merged 2 commits into
trunkfrom
add/stu-1085-studio-display-the-cta-modal-in-the-onboarding-pull-an
Dec 5, 2025
Merged

Display the plans modal in the pull an existing site creation flow#2197
sejas merged 2 commits into
trunkfrom
add/stu-1085-studio-display-the-cta-modal-in-the-onboarding-pull-an

Conversation

@sejas

@sejas sejas commented Dec 5, 2025

Copy link
Copy Markdown
Member

Related issues

Proposed Changes

  • Display NoWpcomSitesModal for new users without any site.
  • Updated the purchase button to don't add studioSiteId when running the pull an existing site creation flow.

Testing Instructions

  • Logout
  • Run npm start
  • Click Add site > Pull an existing site
  • Login or create a new account without any sites
  • Observe that the modal appears
  • Close the modal
  • Go back
  • Repeat
  • Confirm that the modal appears again
  • Remove all sites to display the empty state
  • Confirm that the same behaviour occurs
show-modal-empty-state.mp4
find-a-perfect-plan-when-pulling-an-existing-site.mp4

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@sejas sejas self-assigned this Dec 5, 2025
@sejas
sejas requested review from a team and katinthehatsite December 5, 2025 09:03
@sejas sejas changed the title Add/stu 1085 studio display the cta modal in the onboarding pull an Dec 5, 2025

@katinthehatsite katinthehatsite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in terms of functionality it works fine but I am finding it strange that we display the modal in front of sort of "another modal" with the site search. To me, it would make sense to directly display the information from the NoWpcomSites modal on the screen without having the modal popup when the user logs in. I feel like this would make the design more coherent.

What do you think?

If this needs to be urgently merged before the release, we can do that and then do a follow up to improve the design as an alternative.

@sejas

sejas commented Dec 5, 2025

Copy link
Copy Markdown
Member Author

@katinthehatsite , thanks for the review and for bring it up your concern. I agree that two modals seem a bit strange, although that was the design of @Marinatsu. See p58i-ofU-p2

Screenshot 2025-12-05 at 14 38 34
@github-actions

github-actions Bot commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

📊 Performance Test Results

Comparing 0b08bf9 vs trunk

site-editor

Metric trunk 0b08bf9 Diff Change
load 12707.00 ms 15283.00 ms +2576.00 ms 🔴 20.3%

site-startup

Metric trunk 0b08bf9 Diff Change
siteCreation 15247.00 ms 18569.00 ms +3322.00 ms 🔴 21.8%
siteStartup 5966.00 ms 6961.00 ms +995.00 ms 🔴 16.7%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

@sejas
sejas merged commit d156d79 into trunk Dec 5, 2025
11 of 12 checks passed
@sejas
sejas deleted the add/stu-1085-studio-display-the-cta-modal-in-the-onboarding-pull-an branch December 5, 2025 15:06
@katinthehatsite

Copy link
Copy Markdown
Contributor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants