Skip to content

Conversation

@sejas
Copy link
Member

@sejas sejas commented Dec 4, 2025

Related issues

Proposed Changes

  • Rename Initializing backup... to Initializing remote backup during pull process

Testing Instructions

  • Run npm start
  • Go to the sync tab, connect a site and click on pull
  • Observe the new label
Screenshot 2025-12-04 at 10 51 23

Pre-merge Checklist

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

@sejas sejas self-assigned this Dec 4, 2025
@sejas sejas requested a review from a team December 4, 2025 10:52
Copy link
Contributor

@epeicher epeicher left a comment

Choose a reason for hiding this comment

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

Thanks @sejas, this new message is clearer 👍 . LGTM!

Image

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

📊 Performance Test Results

Comparing 438aa3a vs trunk

site-editor

Metric trunk 438aa3a Diff Change
load 11882.00 ms 12797.00 ms +915.00 ms 🔴 7.7%

site-startup

Metric trunk 438aa3a Diff Change
siteCreation 26876.00 ms 18560.00 ms -8316.00 ms 🟢 -30.9%
siteStartup 8019.00 ms 7951.00 ms -68.00 ms 🟢 -0.8%

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.

The wording looks good 👍

@sejas sejas merged commit 61bd9f3 into trunk Dec 4, 2025
8 of 9 checks passed
@sejas sejas deleted the update/stu-1098-rename-initializing-backup-in-the-pull-process branch December 4, 2025 12:30
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.

4 participants