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

[Regression]: The notification is missing a friend's avatar #208

Closed
Monsterovich opened this issue Dec 5, 2024 · 2 comments · Fixed by #209
Closed

[Regression]: The notification is missing a friend's avatar #208

Monsterovich opened this issue Dec 5, 2024 · 2 comments · Fixed by #209
Labels
bug Bug fix for the user, not a fix to a build script
Milestone

Comments

@Monsterovich
Copy link

Contact Details (e.g. Tox ID or email)

No response

qTox Version

No response

Commit Hash

No response

Toxcore Version

No response

Qt Version

No response

Steps to reproduce

  1. Hide qTox to the background
  2. Receive message from a friend

Observed Behavior

notofication

Expected Behavior

A friend's avatar or an avatar created based on a friend's public key

Reproducibility

Always

Operating System

Linux

Operating System version/distro/name/etc.

Linux Mint

Relevant log output

No response

@Monsterovich Monsterovich added the bug Bug fix for the user, not a fix to a build script label Dec 5, 2024
@iphydf iphydf added this to the v1.18.1 milestone Dec 6, 2024
@iphydf
Copy link
Member

iphydf commented Dec 6, 2024

Yeah that's kind of expected, we got rid of the snore-notify dependency and are now using It's built-in notify. We do pass it an icon, but I guess in some cases that doesn't work. We have plans to add a direct dbus call for notifications to make a better experience for Linux users.

@iphydf
Copy link
Member

iphydf commented Dec 6, 2024

cfbd2bf

Can you test qTox from before this commit? Maybe reverting this commit will work on master. See if that can give you an avatar in the notification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fix for the user, not a fix to a build script
Development

Successfully merging a pull request may close this issue.

2 participants