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

"Sign in via Nextcloud notification" enhancements for macOS #4841

Closed
3 of 5 tasks
jancborchardt opened this issue Aug 10, 2022 · 11 comments
Closed
3 of 5 tasks

"Sign in via Nextcloud notification" enhancements for macOS #4841

jancborchardt opened this issue Aug 10, 2022 · 11 comments
Assignees
Labels
1. to develop design enhancement enhancement of a already implemented feature/code

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Aug 10, 2022

  • The "Sign in via Nextcloud notification" notification does not have a primary button. "Approve" should be that one. :)
  • When clicking either "Approve" or "Deny", the notification should vanish (be marked as read). Currently it stays there
  • It says "Please approve or deny" but there is no "Deny" option. That should be added between "Approve" and "Mark as read"
  • There is a quite long delay in seeing this message. In practice, the Android app displays this immediately, and the desktop client does not, even after waiting 20-30 seconds.
  • Frank reported that on macOS when the message shows the view doesn't scroll up top automatically so you have to scroll manually

Desktop sign in

cc @claucambra

@jancborchardt jancborchardt added 1. to develop design enhancement enhancement of a already implemented feature/code labels Aug 10, 2022
@mbousq
Copy link

mbousq commented Aug 19, 2022

My 2 cts :

  • notifications to be "always-on-top". Can't check right now, but when I had sync-errors I had to scroll down past all the errors to get to the notification. Would be easier have that notification "pinned" somehow 🤔

@camilasan
Copy link
Member

camilasan commented Sep 20, 2022

  • Since 3.6, you can not click on 'Approve' or 'Cancel'

@camilasan
Copy link
Member

camilasan commented Sep 20, 2022

  • It says "Please approve or deny" but there is no "Deny" option. That should be added between "Approve" and "Mark as read"

I think it should change on the server then. The actions provided by the api are only 'Approve' and 'Cancel'. There is no 'Deny'.

@camilasan
Copy link
Member

camilasan commented Sep 20, 2022

[ ] When clicking either "Approve" or "Deny", the notification should vanish (be marked as read). Currently it stays there

this has been fixed.

No.

@camilasan
Copy link
Member

approve
approve-hover

@jancborchardt
Copy link
Member Author

@camilasan btw is there any quite long delay in seeing this message? In practice, the Android app displays this immediately, and the desktop client does not, even after waiting 20-30 seconds.

@camilasan
Copy link
Member

camilasan commented Nov 15, 2023

✅ There seems to be no delays on Linux and the message shows up on top.
linux
🔴 On mac OS however, there is delay.

However, it seems to be inconsistent. I don't always get the same results in both platforms.

I will fix it in the coming days.

@jancborchardt
Copy link
Member Author

@camilasan can confirm, with 3.10.1 it is now basically instant on Ubuntu 20.04. Really amazing, this makes it a looooot more seamless to sign in. :)

@camilasan
Copy link
Member

camilasan commented Nov 20, 2023

Really amazing, this makes it a looooot more seamless to sign in. :)

Thanks for checking on Ubuntu.
Amazing how it is different between platforms, it is beyond me what the difference is 🙈

@camilasan
Copy link
Member

I have no progress on this, I was busy with support.
I also asked for input from Claudio about how mac OS behaves, about the system notifications:
"the system decides when to show the notification unfortunately. We might need to modify our notification system to be less spammy maybe the system is showing it late because it is throttling how many notifications reach the user
(the system does this on macOS)"

But it is unknown at the moment why there is a delay to show it in the activities list (which is platform independent).

@jancborchardt jancborchardt removed their assignment Dec 4, 2023
@camilasan
Copy link
Member

When I was testing the 3.11 release on mac OS Ventura, against cloud.nextcloud.com, there was no delay at all 🤔

2fa-approve

2fa-macos

The exact same notification shows up on the server notifications list and when the user has seen the notification, it should
then get cleared from the desktop client notification list. I am wondering if there is something wrong with that.

@AndyScherzinger AndyScherzinger changed the title "Sign in via Nextcloud notification" enhancements "Sign in via Nextcloud notification" enhancements for macOS Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop design enhancement enhancement of a already implemented feature/code
Projects
Archived in project
Status: 🎉 Done
Development

No branches or pull requests

5 participants