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: invalid parsing of meeting links in description in HTML format #715

Open
jrogov opened this issue Apr 15, 2024 · 0 comments
Open

Bug: invalid parsing of meeting links in description in HTML format #715

jrogov opened this issue Apr 15, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@jrogov
Copy link

jrogov commented Apr 15, 2024

macOS version

14.4.1 (23E224)

MeetingBar version

4.8.0

Installation source

Homebrew

Calendars provider

macOS Calendar app

Bug description

Hello!

First of all, thank you very much for creating MeetingBar! It saves me a lot of time, and is one of the most useful programs I've used so far!

There are 2 issues I found when using MeetingBar to connect to meetings created in Google Calendar (and provided via macOS calendar app): one being technical and one being closer to product decisions, I suppose.

It relates to how MeetingBar prioritizes and parses calendar event fields, what's called "Notes" in macOS Calendar terms, and another one containing Google Meet link.

image

Two issues are:

  1. When connecting/copying link to event, MeetingBar prefers using the link in Notes section, instead of the actual Google Meet link (so for example above it would open https://google.com instead)
  2. If Notes field contains some HTML content (which Google Calendar allows to put there), MeetingBar copies the whole section, making it impossible to connect to meeting altogether, since it tries to open something like <ul><li>list item<br></li></ul><br><a href="https://google.com" target="_blank">https://google.com</a> (confirmed by "Copy meeting link" function of MeetingBar), which is, obviously, not a valid URL, leading to the errors below
image

There seems to be a problem with how MeetingBar works with events created in Google Calendar and, specifically, containing HTML content in URL/Notes field.

Whenever event contains a Google Meet link in location field AND a link in event's URL/Notes field that's formatted in HTML

Reproduction steps

  1. Open Google Calendar
  2. Create an event (see pic below) that will contain:
    1. Google Meet link, generated and provided by Google Calendar itself
    2. HTML content, for example consisting of a single list item and another link itself
SCR-20240415-rmm
  1. Sync macOS Calendar app
  2. Try to connect to the meeting via MeetingBar to witness a failure, OR
  3. Use "Copy meeting link" to see that the whole event's whole Notes contents were copied (and used in previous step)

Expected behavior

For first issue, I would argue that in particular for Google Calendar/Meet case it makes sense to use the Google Meet link provided in event description, ignoring Notes content altogether.

For second issue, If the parsing issue spreads to other calendar providers as well, it makes sense to fix parsing as a whole.

Screenshots/screen recordings

Provided above

Additional context

I believe I've included the whole context required, but feel free to reach out in case I've missed anything :)

@jrogov jrogov added the bug Something isn't working label Apr 15, 2024
@jrogov jrogov changed the title Bug: Bug: invalid parsing of meeting links in description in HTML format May 10, 2024
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