From ff5156f36ace8f97edfae528f555256aab1db0f9 Mon Sep 17 00:00:00 2001 From: c0mrade Date: Tue, 26 May 2026 14:04:47 +0300 Subject: [PATCH] refactor(routing): drop now-redundant per-route ErrorBoundary wrappers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Six routes were wrapped in around LazyPage. After the previous commit moved a level='page' boundary inside LazyPage, the outer 'app' boundary became unreachable — the inner one catches first. Strip the dead wrappers (PurchaseSuccess, QuickPurchase, AutoLogin, TopUpResult, GiftSubscription, GiftResult). Behavior is identical, the route blocks are 25 fewer lines, and there's only one boundary to reason about per route. --- src/App.tsx | 56 +++++++++++++++++++++-------------------------------- 1 file changed, 22 insertions(+), 34 deletions(-) diff --git a/src/App.tsx b/src/App.tsx index 5ad2288..589781d 100644 --- a/src/App.tsx +++ b/src/App.tsx @@ -271,31 +271,25 @@ function App() { - - - - + + + } /> - - - - + + + } /> - - - - + + + } /> @@ -394,11 +388,9 @@ function App() { path="/balance/top-up/result" element={ - - - - - + + + } /> @@ -525,25 +517,21 @@ function App() { - - - - - - + + + + + } /> - - - - - - + + + + + } />