Skip to content

Add first-class location and online_url for hybrid events - #204

Closed
edalzell wants to merge 1 commit into
mainfrom
feature/location-online-url
Closed

edalzell wants to merge 1 commit into
mainfrom
feature/location-online-url

Conversation

@edalzell

@edalzell edalzell commented Sep 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Declare location, online_url, and coordinates on the event fieldset so physical, online, and hybrid events are first-class
  • Replace address/link URL sniffing with explicit ICS mapping (decorate / icsLocation / icsUrl), including the multi-day whole-event download gap
  • Add a 7.0 MigrateLocationFields update script plus docs/tests for the mapping table and migration edge cases

Test plan

  • vendor/bin/pest
  • Create physical-only, online-only, and hybrid events; download ICS and verify LOCATION: / URL: / GEO:
  • Confirm online-only join link is clickable in Calendar.app
  • Download a multi-day event without a date param and confirm location/URL/description are present
  • Run the 7.0 update path against sample address / link / URL-location entries (and confirm array-shaped location is left alone)

Declare the fields, map them unambiguously into ICS, and migrate old address/link sniffing via a 7.0 update script.
@edalzell

edalzell commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

Re-cutting into the stepped PRs from #190 (#191–#196) instead of one monolithic change. Closing this draft; work continues on those issues.

@edalzell edalzell closed this Sep 8, 2026
@edalzell
edalzell deleted the feature/location-online-url branch September 8, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant