Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to disable connection error notifications #6517

Closed
r33int opened this issue Mar 7, 2024 · 13 comments
Closed

Ability to disable connection error notifications #6517

r33int opened this issue Mar 7, 2024 · 13 comments
Assignees
Labels
design enhancement enhancement of a already implemented feature/code

Comments

@r33int
Copy link

r33int commented Mar 7, 2024

How to use GitHub

  • Please use the 👍 reaction to show that you want to have the same feature implemented.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Feature description

Since the latest Nextcloud Desktop update, I keep getting notifications about connection errors (Connection timed out, Host not found) when my laptop is offline. These notifications pop every few minutes.

As it is expected to have connection errors when offline, it is pretty annoying for me that Nextcloud reminds me every few minutes that I'm offline.

I tried turning off server notifications, but it did not suppress the connection notifications. Are there any plans to add the ability to turn off these notifications?

image

@r33int r33int added the enhancement enhancement of a already implemented feature/code label Mar 7, 2024
@rdsherwood
Copy link

I have encountered the same problem, and I agree, there should be an option to disable them.

In my case, the error message displays when I wake the computer from sleep, because it takes a few seconds to reconnect to the network. If you could delay the error for 5 or 10 seconds from awakening or from a restart, it would resolve the problem.

@chiva
Copy link

chiva commented Mar 8, 2024

Similar here

I get connected to my Nextcloud instance by Tailscale, but also use it for work, so I have to switch profiles on Tailscale.
This means I'm a lot of hours not connected to my home VPN and getting error notifications every minute, very annoying.

I see it added in PR 6438
Specifically from 35108a8

Can't see any explanation on why was it pushed, more so when it degrades UX

I'd understand this state to be shown in the Nextcloud taskbar icon with a warning icon or something like that, and the actual message within the deployed window when clicking on it.

@maddentim
Copy link

It should at least have a little patience. When my laptop wakes from suspend, Nextcloud client throws a network not found error immediately even though the network connection was in the process of reconnecting. If I suspend a resume my system a half a dozen times at, these useless errors just pile up. Oof.

@camilasan
Copy link
Member

camilasan commented Mar 8, 2024

Thanks for the feedback, it should be possible to make them more 'patience'.

wdyt @nextcloud/designers ?

@camilasan camilasan self-assigned this Mar 8, 2024
@nimishavijay
Copy link
Member

@camilasan agreed, that can be annoying.

We could delay the connection check by a few minutes after starting up, if that's possible to detect? So if there is no connection found 2 mins after starting up, the error can be shown. Would that be possible? :)

A good error message also has some suggested actions that the user can take, even if it's a simple "Please try again later". That would help troubleshoot the issues if they are happening frequently.

If the error message is really annoying I'm also thinking of having a "Don't show again" checkbox which would stop the notification popping up until the next startup. But this sounds risky, what do you think?

@maddentim
Copy link

Personally, I do not see this notification ever being particularly useful to me. I definitely do not need the nextcloud client to tell me it could not connect to the server when the client system has no internet. Maybe if the system can reach the internet but the server was still inaccessible, I suppose that could be useful. Can you imagine if every client application on your system popped a notification every time could not connect?

@r33int
Copy link
Author

r33int commented Mar 8, 2024

I'd suggest to turn off (or at least add an option) these notifications for connection errors that are most likely to be on the users's network side (e.g not connected to any network, timed out), as the user is likely already aware of these issues and does not need Nextcloud to notify them.

I'd also mute time out notifications, as these notifications can appear a lot when using an unstable connection (e.g when riding a train), and again, the user is most likely already aware of that.

In these cases, I would show an info box in the systray pane saying for example "You're currently offline. Syncing is paused and will continue when you go online again."

These notifications would indeed be very useful for unexpected issues related to the server itself (internal error, invalid certificate etc).

For most users, Nextcloud client just simply sits there in the background and does its job at syncing, and having constant notifications from it can get disturbing.

That's my own opinion on this, let me know what you think of it!

@Nydragon
Copy link

My Nextcloud instance is behind a VPN, and as I am not connected to it most of the time I get this notification every ~6 minutes. I don't think that this is a rare use case and so I am not sure if making Nextcloud more patient by a couple of minutes after coming out of sleep properly addresses this issue. For me waiting multiple hours (6, 24?) and being able to disable those notifications would make more sense

@jancborchardt
Copy link
Member

I definitely do not need the nextcloud client to tell me it could not connect to the server when the client system has no internet. Maybe if the system can reach the internet but the server was still inaccessible, I suppose that could be useful. Can you imagine if every client application on your system popped a notification every time could not connect?

Agree with @maddentim’s take.

@camilasan there should be no need for a notification if the whole system is offline anyway – other applications do not send a notification on that either.
We already change the tray icon to an offline icon (like other apps), and we change the state to "Offline" in the tray menu. That is enough as info, is it not?

But no notification should be needed, and especially not a setting to either enable or disable them.

@perniczech
Copy link

This notification is REALLY annoying. IF someone does actually need it then have it toggleable in the settings but I don't need an app to pester me about its inability to connect to a server all the time when I'm not even connected to the internet.

@camilasan
Copy link
Member

It has been fixed in #6550
It is released in 3.12.2

@Slater91
Copy link

The issue has not been fixed. The notifications keep on appearing. Using Nextcloud 3.12.3 on Linux.

@mike2307
Copy link

It has been fixed in #6550 It is released in 3.12.2

No, it is definitely NOT fixed.
@camilasan: Please reopen this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design enhancement enhancement of a already implemented feature/code
Projects
None yet
Development

No branches or pull requests