leits / MeetingBar

🇺🇦 Your meetings at your fingertips in the macOS menu bar
https://meetingbar.app
Apache License 2.0
4.54k stars 203 forks source link

Bug: Fullscreen notifications not displayed for short events in quick succession #769

Open leochab opened 1 month ago

leochab commented 1 month ago

macOS version

14.6.1 (23G93)

MeetingBar version

4.10.0

Installation source

manually

Calendars provider

Google Calendar

Bug description

I have configured MeetingBar to display a fullscreen notification when an event starts, as shown below:

MeetingBar Notification Setup

However, I am experiencing an issue where I do not receive a fullscreen notification for the first meeting when there are two short meetings scheduled in quick succession. The meetings do not overlap; for example:

In this scenario, I do not receive any notification for the first meeting. It's worth noting that the first meeting is a recurring event, and the notification works correctly on other days of the week when there are no subsequent meetings scheduled.

Example of Meetings

Additional Information:

Reproduction steps

  1. Schedule two short meetings (< 15 minutes) in quick succession (e.g., one ends just before the next starts).

Expected behavior

Both events generate a fullscreen notification

Screenshots/screen recordings

No response

Additional context

Note that two 15-minutes meetings in succession work. For instance:

dosubot[bot] commented 1 month ago

Hey @leochab 👋, I'm here to help you with your MeetingBar issue! I can assist with bug fixes, answer questions, and guide you on becoming a contributor. Let's tackle this challenge together!

The issue you're experiencing with fullscreen notifications not being displayed for short events in quick succession, particularly when the first event is recurring, might be related to how MeetingBar processes and schedules notifications.

MeetingBar uses a timer that checks for upcoming events every 10 seconds to determine if a fullscreen notification should be displayed. It filters out past events and checks if the fullscreen notification feature is active. For non-all-day events, it considers displaying a notification if the event is within a specific time interval before the event start time [1].

However, there might be a limitation or condition in the code that affects how notifications are handled for short-duration events, especially when they are recurring and scheduled closely with other events. Unfortunately, there isn't specific documentation or known issues in the codebase regarding notifications for short-duration events or how they are prioritized when scheduled in quick succession [1].

Given this, it might be worth checking if there are any settings or conditions that could be suppressing notifications for recurring events in your specific setup. Additionally, ensuring that the notification settings are correctly configured and that the alert style is set to allow fullscreen notifications might help.

To continue talking to Dosu, mention @dosu.