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

No sounds on notifications #33

Closed
tom79 opened this issue Mar 4, 2021 · 4 comments
Closed

No sounds on notifications #33

tom79 opened this issue Mar 4, 2021 · 4 comments

Comments

@tom79
Copy link

tom79 commented Mar 4, 2021

Your app is excellent and I commend you on creating it. I only wish I'd found it earlier!

There is one issue I've discovered and it's that despite enabling notifications and receiving a banner alert for new emails, there is no sound despite the "Play sound for notifications" box being checked in System Preferences/Notifications. While the macOS notification system does support the playing of sounds, I believe the issue is that your app needs to "provide" a sound to the operating system. I get sound notifications for all my other apps with no problem.

Aside from this, the app is excellent and frankly I don't know why Fastmail themselves haven't created an official version. I really dislike using web/browser-based apps particularly for something as frequently used as email. Your wrapper is a very neat solution to what is an excellent email provider and an excellent web interface. Thank you!

EDIT : I am using the latest version of macOS Big Sur, version 11.2.1, in case that is helpful.

@joelekstrom
Copy link
Owner

Hi! Thanks for the report, and sorry for the slow response! I’ll take a look at this.

I’m glad you’re enjoying Fastmate! Thanks for using it 😊

@joelekstrom
Copy link
Owner

Hi again, this was a total oversight on my part. I forgot to add the sound flag when adding support for the "modern" notification system used on 10.14+. 😅 Fixed in v1.6.5

Thanks again for the report!

@tom79
Copy link
Author

tom79 commented Mar 13, 2021 via email

@joelekstrom
Copy link
Owner

It was a whopping single line of code added!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants