[PR #56] [MERGED] fix: resolve UI freeze during notification processing and startup crash #57

New issue

Closed

opened 2026-03-04 23:13:45 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 23:13:45 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/creamlike1024/EasyLPAC/pull/56
Author: @smanlin
Created: 2/24/2026
Status: ✅ Merged
Merged: 2/24/2026
Merged by: @creamlike1024

Base: master ← Head: fix/notification-freeze

📝 Commits (3)

5ab8438 fix: resolve UI freeze and thread safety issues during notification processing
3a624cb fix: wrap dialog calls in fyne.Do to ensure thread safety
d064213 fix: provide valid empty JSON for EUM registry to prevent startup crash

📊 Changes

4 files changed (+186 additions, -163 deletions)

View changed files

📝 control.go (+75 -65)
➕ eum-registry.json (+1 -0)
📝 widgets.go (+102 -90)
📝 window.go (+8 -8)

📄 Description

This PR resolves the UI freeze and thread safety issues encountered when processing notifications.

Key Changes:

Responsiveness: Moved batch notification processing to background tasks to prevent UI blocking.
Thread Safety: Wrapped UI updates and dialog calls in fyne.Do() to ensure correct execution context and resolve thread-related errors.
Stability: Fixed a startup panic caused by an invalid EUM registry file.

_{🔄 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/creamlike1024/EasyLPAC/pull/56 **Author:** [@smanlin](https://github.com/smanlin) **Created:** 2/24/2026 **Status:** ✅ Merged **Merged:** 2/24/2026 **Merged by:** [@creamlike1024](https://github.com/creamlike1024) **Base:** `master` ← **Head:** `fix/notification-freeze` --- ### 📝 Commits (3) - [`5ab8438`](https://github.com/creamlike1024/EasyLPAC/commit/5ab843882b4accc576966968631a8591220ba353) fix: resolve UI freeze and thread safety issues during notification processing - [`3a624cb`](https://github.com/creamlike1024/EasyLPAC/commit/3a624cb454dd60eb2ac0405cffa146ecfeadbe0f) fix: wrap dialog calls in fyne.Do to ensure thread safety - [`d064213`](https://github.com/creamlike1024/EasyLPAC/commit/d06421324cdde464cbd1a66e53b86bdc2e481d1f) fix: provide valid empty JSON for EUM registry to prevent startup crash ### 📊 Changes **4 files changed** (+186 additions, -163 deletions) <details> <summary>View changed files</summary> 📝 `control.go` (+75 -65) ➕ `eum-registry.json` (+1 -0) 📝 `widgets.go` (+102 -90) 📝 `window.go` (+8 -8) </details> ### 📄 Description This PR resolves the UI freeze and thread safety issues encountered when processing notifications. ### Key Changes: - **Responsiveness**: Moved batch notification processing to background tasks to prevent UI blocking. - **Thread Safety**: Wrapped UI updates and dialog calls in `fyne.Do()` to ensure correct execution context and resolve thread-related errors. - **Stability**: Fixed a startup panic caused by an invalid EUM registry file. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>