Skip to content

chore: render errors as content (preserving nav, footer, etc.) + redirect fix#16906

Merged
mzikherman merged 2 commits intomainfrom
revert-16905-revert-16897-content-error-boundary
Mar 10, 2026
Merged

chore: render errors as content (preserving nav, footer, etc.) + redirect fix#16906
mzikherman merged 2 commits intomainfrom
revert-16905-revert-16897-content-error-boundary

Conversation

@mzikherman
Copy link
Contributor

Reverting the revert, + a commit to fix the issue!

Everything from #16897 is re-instated!

@mzikherman mzikherman requested review from ansor4 and starsirius March 6, 2026 21:38
@mzikherman mzikherman self-assigned this Mar 6, 2026
@relativeci
Copy link

relativeci bot commented Mar 6, 2026

#7096 Bundle Size — 8.66MiB (+0.03%).

9374cc0(current) vs 9234a95 main#7093(baseline)

Warning

Bundle contains 31 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes Regression 1 regression
                 Current
#7096
     Baseline
#7093
Regression  Initial JS 2.84MiB(+0.09%) 2.84MiB
No change  Initial CSS 0B 0B
No change  Cache Invalidation 19.5% 19.5%
No change  Chunks 110 110
No change  Assets 113 113
Change  Modules 5522(+0.04%) 5520
No change  Duplicate Modules 618 618
No change  Duplicate Code 4.93% 4.93%
No change  Packages 250 250
No change  Duplicate Packages 30 30
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#7096
     Baseline
#7093
Regression  JS 8.51MiB (+0.03%) 8.51MiB
No change  Other 152.75KiB 152.75KiB

Bundle analysis reportBranch revert-16905-revert-16897-conten...Project dashboard


Generated by RelativeCIDocumentationReport issue

// Skip Found Redirect instances — spreading them into plain objects
// destroys their prototype `render()` method, breaking the redirect.
if (route instanceof Redirect) {
return route
Copy link
Member

Choose a reason for hiding this comment

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

🎉

Copy link
Member

@ansor4 ansor4 left a comment

Choose a reason for hiding this comment

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

nice!

@mzikherman
Copy link
Contributor Author

Re-trying this!

@mzikherman mzikherman merged commit 6e64382 into main Mar 10, 2026
13 checks passed
@mzikherman mzikherman deleted the revert-16905-revert-16897-content-error-boundary branch March 10, 2026 14:28
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