teams-for-linux icon indicating copy to clipboard operation
teams-for-linux copied to clipboard

No meeting notifications in KDE

Open mrkaczor83 opened this issue 3 years ago • 5 comments

Describe the bug I am receiving chat notifications but no meeting notifications. "'--enableDesktopNotificationsHack" doesn't help. Same behavior.

Also there is no sound notifications only KDE tray ones for all the events although I have turned them on.

The second issue might be related to https://github.com/wavebox/waveboxapp/issues/418

To Reproduce Steps to reproduce the behavior:

  1. Setup notification for all events – under settings, notifications
  2. No notification for meetings

Expected behavior "Meetings start" notification like chat message one

Desktop (please complete the following information):

  • OS: Debian 11 on KDE
  • Installation package deb
  • Version 1.0.12

mrkaczor83 avatar Oct 19 '21 14:10 mrkaczor83

Hi, thanks for reporting.

Yes, the notification is not implemented as for what it says in the wavebox link. You need to implement an extra logic.

There is also a really old open bug from MS about notification for meetings. https://techcommunity.microsoft.com/t5/microsoft-teams/audio-notifications/m-p/386653

I suspect the enable desktop notification hack might be broken. We did move the version of electron and I don't think that was tested.

Probably check if you can get it to work using what they recommend on the wavebox link (commands) and if you do we can try to port that into the app.

Thanks

IsmaelMartinez avatar Oct 22 '21 06:10 IsmaelMartinez

There is no sound because of electron implementation ... but maybe we can have simple GUI notification like for chat? Like "Meeting has started" or any other that will have similar value

mrkaczor83 avatar Nov 24 '21 15:11 mrkaczor83

can you check if this is still the case in 1.0.24? thanks!

IsmaelMartinez avatar Mar 11 '22 10:03 IsmaelMartinez

@IsmaelMartinez Yes, this is still an issue, in my case on Ubuntu Gnome 20.04 with version 1.0.24 installed from snap. Other notification types (like calls and chat notifications) are working, but meeting notifications are not.

floscher avatar Mar 11 '22 11:03 floscher

I'm using Bspwm with dunst for notifications and I'm facing the same issue. With the official client, message notifications show as native dunst ones, but meetings and calls as custom team ones. Just using the web client in any browser acts like this client, not showing meeting notifications at all. Which makes me think it was just not implemented on microsoft's side and this project probably can't do much about it... :cry:

opalmay avatar May 26 '22 07:05 opalmay

Hi there! Do you know when Microsoft will implement this feature? It's very annoying to no have this notification, as we have in the client. Thank you for all your effort in this project :)

danielgsanz avatar Oct 01 '22 08:10 danielgsanz

Hi there! Do you know when Microsoft will implement this feature? It's very annoying to no have this notification, as we have in the client. Thank you for all your effort in this project :)

I would hope this year, because the official electron client is being deprecated in favor of the web app becoming a PWA. But there's no knowing with Microsoft

opalmay avatar Oct 01 '22 13:10 opalmay

Hi there! Do you know when Microsoft will implement this feature? It's very annoying to no have this notification, as we have in the client. Thank you for all your effort in this project :)

I'm breaking my head over this. 😁 Need to reverse engineer the js implementation from MS. Tired!

I wonder when they release pwa what happens to Mac and Windows clients. It could possibly break this app too. Only time will tell.

jijojosephk avatar Oct 02 '22 14:10 jijojosephk

Looks like the communication is happening through websocket. Unable to debug or find the event that updates the meeting details.

jijojosephk avatar Oct 08 '22 16:10 jijojosephk

Hooray! Finally, I've found something. It will come out soon. :smile:

jijojosephk avatar Oct 09 '22 11:10 jijojosephk

Fixed in 1.0.44. Re-open if it's not fixed.

jijojosephk avatar Oct 09 '22 12:10 jijojosephk

Unfortunately, the problem still persists for me in teams-for-linux v1.0.44 Flatpak.

System details

Operating System: Kubuntu 22.04 KDE Plasma Version: 5.25.5 KDE Frameworks Version: 5.98.0 Qt Version: 5.15.3 Kernel Version: 5.15.0-48-generic (64-bit) Graphics Platform: X11

FloEdelmann avatar Oct 11 '22 08:10 FloEdelmann

I had tested this on Arch KDE. Plasma should be same for all right? Haven't tested on Kubuntu. Can you confirm if this is the scenario.

  1. There's a meeting on your calendar
  2. Someone starts the meeting.
  3. You should see that 'Meeting started'

jijojosephk avatar Oct 11 '22 09:10 jijojosephk

O boy, o boy, o boy, I will test it tomorrow :) thanks!

mrkaczor83 avatar Oct 11 '22 09:10 mrkaczor83

This is how it looks on Ubuntu Budgie. With KDE and the default setup, it should show bottom right.

image

jijojosephk avatar Oct 11 '22 09:10 jijojosephk

It's working for me in KDE with the latest version. However, I keep receiving meeting popups when I am not expecting them (I have no meetings scheduled at that time). Would it be possible to at least display the name of the meeting in the popup?

cs96and avatar Oct 11 '22 09:10 cs96and

@jijojosephk Yes, the scenario is exactly as you described. Note that it is a recurring meeting; maybe that makes a difference?

FloEdelmann avatar Oct 11 '22 09:10 FloEdelmann

@cs96and that's an issue. The logic does not validate the date and time. Purely based on the serviceworker message push. I've seen the problem too. But at least now we know what can be done. I had to go through each and every service objects written by Microsoft to find which event triggers for the notification. Was a herculean task.

jijojosephk avatar Oct 11 '22 09:10 jijojosephk

@cs96and can you check this when version 1.0.45 is out?

jijojosephk avatar Oct 11 '22 17:10 jijojosephk

@cs96and Meeting name also will be displayed.

jijojosephk avatar Oct 11 '22 17:10 jijojosephk

Is this in the pre-release 1.0.45, or do I need to wait for a proper 1.0.45?

cs96and avatar Oct 12 '22 12:10 cs96and

@cs96and it's official release now. snap, flatpak, and here. you may try.

jijojosephk avatar Oct 13 '22 04:10 jijojosephk

Sorry, I didn't try it with the --enableDesktopNotificationsHack CLI option. With that enabled, it works now. Thanks for fixing!

FloEdelmann avatar Oct 13 '22 08:10 FloEdelmann

@FloEdelmann This change has no connection with the flag. I wonder what's the issue at your side. Still can't figure out.

jijojosephk avatar Oct 13 '22 08:10 jijojosephk

If there's anything I can try, let me know.

FloEdelmann avatar Oct 13 '22 09:10 FloEdelmann

@FloEdelmann try maybe send any notification from CLI, for me all works perfectly but I have no big teams so additional notifications are not an issue so much :)

mrkaczor83 avatar Oct 13 '22 09:10 mrkaczor83

@mrkaczor83 I don't know what you mean with sending notifications from the CLI. Meeting notifications are working fine for me now with the --enableDesktopNotificationsHack flag enabled. So from my side, everything is fine; but apparently it should also be fixed without the flag.

FloEdelmann avatar Oct 13 '22 09:10 FloEdelmann

Aa sorry, I haven't notice your last comment

mrkaczor83 avatar Oct 13 '22 09:10 mrkaczor83