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

Bug: false warning for notification settings #777

Open
roeniss opened this issue Nov 13, 2024 · 2 comments
Open

Bug: false warning for notification settings #777

roeniss opened this issue Nov 13, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@roeniss
Copy link

roeniss commented Nov 13, 2024

macOS version

14.6.1(23G93)

MeetingBar version

4.10.0

Installation source

Homebrew

Calendars provider

Google Calendar

Bug description

  • I allowed notification of MeetingBar.
  • It works perfectly.
  • The warning is still there: "Your macOS notification settings for MeetingBar are not set to alert. Do so if you want persistent notifications."

Reproduction steps

I found this bug right after installation.

Expected behavior

The warning disappears.

Screenshots/screen recordings

sceenshot--2024-11-13--14-49-57@2x

Additional context

I'm struggling with another issue that a fullscreen notification seems not working (I'll report that separately). This may be related to it.

@roeniss roeniss added the bug Something isn't working label Nov 13, 2024
Copy link

dosubot bot commented Nov 13, 2024

A similar issue was previously reported and closed: Bug: Notification warning overlaps with notification setting. The solution mentioned was that it was a bug in the SwiftUI view, and it was to be fixed in the next release at that time [1].

In the current codebase, the logic for triggering the notification settings warning is in the JoinEventNotificationPicker struct. It uses the checkNotificationSettings() function to determine the status. The warning is displayed if the alert style is not set to alert and notifications are not disabled, or if notifications are disabled [2].

If you have already allowed notifications and the warning persists, it might be worth checking if the alert style is correctly set to alert or banner in your macOS notification settings for MeetingBar. If the issue continues, it could be a regression or a new bug that might need further investigation.

To continue talking to Dosu, mention @dosu.

1 similar comment
Copy link

dosubot bot commented Nov 13, 2024

A similar issue was previously reported and closed: Bug: Notification warning overlaps with notification setting. The solution mentioned was that it was a bug in the SwiftUI view, and it was to be fixed in the next release at that time [1].

In the current codebase, the logic for triggering the notification settings warning is in the JoinEventNotificationPicker struct. It uses the checkNotificationSettings() function to determine the status. The warning is displayed if the alert style is not set to alert and notifications are not disabled, or if notifications are disabled [2].

If you have already allowed notifications and the warning persists, it might be worth checking if the alert style is correctly set to alert or banner in your macOS notification settings for MeetingBar. If the issue continues, it could be a regression or a new bug that might need further investigation.

To continue talking to Dosu, mention @dosu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants