Skip to content

Conversation

@sejas
Copy link
Member

@sejas sejas commented Dec 2, 2025

Related issues

Proposed Changes

  • Remove streamlineOnboarding feature flag
  • It also removes the Publish and Pull site buttons in favor of always displaying the connect site button.

Testing Instructions

  • Run npm start
  • Create a site
  • Go to the sync tab
  • Observe that the connect a site button is visible
  • Also observe that the Publish site button next to the start button is present
Screenshot 2025-12-02 at 17 41 43

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@sejas sejas self-assigned this Dec 2, 2025
@sejas sejas requested a review from a team December 2, 2025 17:42
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

📊 Performance Test Results

Comparing 7eab339 vs trunk

site-editor

Metric trunk 7eab339 Diff Change
load 17021.00 ms 13899.00 ms -3122.00 ms 🟢 -18.3%

site-startup

Metric trunk 7eab339 Diff Change
siteCreation 24707.00 ms 24995.00 ms +288.00 ms 🔴 1.2%
siteStartup 10066.00 ms 10083.00 ms +17.00 ms 🔴 0.2%

Results are median values from multiple test runs.

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

Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

Choose a reason for hiding this comment

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

Looks good to me although I see some unit tests are failing so might need to be fixed 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants