[PR #2657] [MERGED] improving error handling for lazy-loading #66804

Closed
opened 2026-05-14 01:08:41 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2657
Author: @MatissJanis
Created: 4/23/2024
Status: Merged
Merged: 4/25/2024
Merged by: @MatissJanis

Base: masterHead: matiss/lazy-load


📝 Commits (4)

  • e193412 improving error handling for lazy-loading
  • 5bcb658 Release notes
  • 6dd311f Feedback: loading title
  • 10ba7dc Merge branch 'master' into matiss/lazy-load

📊 Changes

7 files changed (+85 additions, -27 deletions)

View changed files

📝 packages/desktop-client/src/components/App.tsx (+1 -1)
📝 packages/desktop-client/src/components/FatalError.tsx (+38 -14)
📝 packages/desktop-client/src/components/common/Modal.tsx (+10 -8)
📝 packages/desktop-client/src/components/gocardless/GoCardlessLink.tsx (+1 -1)
📝 packages/desktop-client/src/components/util/LoadComponent.tsx (+19 -3)
📝 packages/loot-core/src/shared/errors.ts (+10 -0)
upcoming-release-notes/2657.md (+6 -0)

📄 Description

Added a custom error message if lazy-loading fails.

Screenshot 2024-04-23 at 17 55 32

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/2657 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 4/23/2024 **Status:** ✅ Merged **Merged:** 4/25/2024 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/lazy-load` --- ### 📝 Commits (4) - [`e193412`](https://github.com/actualbudget/actual/commit/e193412ca0ef8c08547b47edbfbda6b830ce7251) :sparkles: improving error handling for lazy-loading - [`5bcb658`](https://github.com/actualbudget/actual/commit/5bcb658f575b3a65d9d785b9880c6db0cd18a599) Release notes - [`6dd311f`](https://github.com/actualbudget/actual/commit/6dd311f12fcc9e5d166e3f1d35d10c00ea590a19) Feedback: loading title - [`10ba7dc`](https://github.com/actualbudget/actual/commit/10ba7dc3069d3cf6e74f937de2a95eecacf8c34d) Merge branch 'master' into matiss/lazy-load ### 📊 Changes **7 files changed** (+85 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/App.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/FatalError.tsx` (+38 -14) 📝 `packages/desktop-client/src/components/common/Modal.tsx` (+10 -8) 📝 `packages/desktop-client/src/components/gocardless/GoCardlessLink.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/util/LoadComponent.tsx` (+19 -3) 📝 `packages/loot-core/src/shared/errors.ts` (+10 -0) ➕ `upcoming-release-notes/2657.md` (+6 -0) </details> ### 📄 Description Added a custom error message if lazy-loading fails. <img width="615" alt="Screenshot 2024-04-23 at 17 55 32" src="https://github.com/actualbudget/actual/assets/886567/5b13abd3-7ec8-4cc9-b9b4-ce93c8835afd"> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-14 01:08:41 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#66804