[PR #4751] client/ui: more understandable connection lost messages #26217

Open
opened 2026-08-05 07:07:02 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4751
Author: @nazarewk
Created: 11/5/2025
Status: 🔄 Open

Base: mainHead: feat/better-daemon-connection-lost-message


📝 Commits (1)

  • ec4d092 client/ui: more understandable connection lost messages

📊 Changes

1 file changed (+26 additions, -5 deletions)

View changed files

📝 client/ui/client_ui.go (+26 -5)

📄 Description

Describe your changes

The existing message is misleading and alarming for customers, even though the "error" is supposed to recover on the next run.

WARNING: I don't know how to cause this type of failure to test it in NetBird. I have generated the same message in https://github.com/fyne-io/demo to confirm fyne support multi-line notificatinos.

N/A

Stack

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 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/netbirdio/netbird/pull/4751 **Author:** [@nazarewk](https://github.com/nazarewk) **Created:** 11/5/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feat/better-daemon-connection-lost-message` --- ### 📝 Commits (1) - [`ec4d092`](https://github.com/netbirdio/netbird/commit/ec4d09215c956909fc691110585ebd75ffa28264) client/ui: more understandable connection lost messages ### 📊 Changes **1 file changed** (+26 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `client/ui/client_ui.go` (+26 -5) </details> ### 📄 Description ## Describe your changes The existing message is misleading and alarming for customers, even though the "error" is supposed to recover on the next run. WARNING: I don't know how to cause this type of failure to test it in NetBird. I have generated the same message in https://github.com/fyne-io/demo to confirm `fyne` support multi-line notificatinos. ## Issue ticket number and link N/A ## Stack <!-- branch-stack --> ### Checklist - [x] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 07:07:02 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#26217