Skip to content

Studio: Improve error handling when importing invalid file formats - #2202

Merged
katinthehatsite merged 3 commits into
trunkfrom
fix/dragging-invalid-file-experience
Dec 9, 2025
Merged

Studio: Improve error handling when importing invalid file formats#2202
katinthehatsite merged 3 commits into
trunkfrom
fix/dragging-invalid-file-experience

Conversation

@katinthehatsite

Copy link
Copy Markdown
Contributor

Related issues

Fixes STU-1050

Proposed Changes

This PR ensures that when the invalid file format is added for import, we display an error in the Add site modal:

Screenshot 2025-12-05 at 2 16 29 PM

Additionally, it moves file validation to an earlier point in the Import process and also displays an error to the user instead of starting the process and then throwing an error:

Screenshot 2025-12-05 at 2 16 04 PM

Testing Instructions

  • Pull the changes from this branch
  • Start Studio with npm start
  • Click on the Add site in the sidebar
  • Select Import from backup option
  • Drag an invalid file format like .png
  • Observe that you can see an error appearing
  • Navigate back
  • Click again on Import from backup option
  • Confirm that the error is gone
  • Navigate to the Import/Export tab
  • Drag an invalid file format into the field
  • Observe that you can see the error
  • Switch to other tabs and then switch back to Import/Export tab
  • Observe that the error is cleared

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@katinthehatsite katinthehatsite self-assigned this Dec 5, 2025
@katinthehatsite
katinthehatsite requested a review from a team December 5, 2025 13:41
@github-actions

github-actions Bot commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

📊 Performance Test Results

Comparing d1a3d66 vs trunk

site-editor

Metric trunk d1a3d66 Diff Change
load 10261.00 ms 15335.00 ms +5074.00 ms 🔴 49.4%

site-startup

Metric trunk d1a3d66 Diff Change
siteCreation 29018.00 ms 31271.00 ms +2253.00 ms 🔴 7.8%
siteStartup 11122.00 ms 10075.00 ms -1047.00 ms 🟢 -9.4%

Results are median values from multiple test runs.

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

@gcsecsey gcsecsey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks Kat for improving the error messages, the new error styling is informative and minimal IMO, it works great. I confirm that the errors are also cleared when navigating away and back again.

Import from a backup Import/Export tab
Image Image
@katinthehatsite
katinthehatsite merged commit 646d70d into trunk Dec 9, 2025
9 checks passed
@katinthehatsite
katinthehatsite deleted the fix/dragging-invalid-file-experience branch December 9, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants