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

Fix crash on startup due to events with nil email #461

Merged
merged 3 commits into from
May 23, 2022
Merged

Fix crash on startup due to events with nil email #461

merged 3 commits into from
May 23, 2022

Conversation

amcedwards
Copy link
Contributor

@amcedwards amcedwards commented May 23, 2022

Status

READY

Description

Small fix for #460: Created EKParticipant.safeURL as EKParticipant.url is not an optional.
(Note: I'm brand new to Swift, so am quite open to a better way to solve this!)

Checklist

Steps to Test or Reproduce

  • Create a meeting request (via Outlook/O365) with an attendee that has an X500 email address.
  • Review the meeting in Calendar; note yellow exclamation mark next to the attendee with an invalid address
  • Launch MeetingBar

@leits
Copy link
Owner

leits commented May 23, 2022

WOW
Thank you, @amcedwards!!

I spent so much time trying to fix this bug. Too much. 😓

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

Successfully merging this pull request may close these issues.

2 participants