Skip to content

Add nested location group; remove address, link, and sniffing - #217

Merged
edalzell merged 1 commit into
mainfrom
feature/location-field
Sep 9, 2026
Merged

edalzell merged 1 commit into
mainfrom
feature/location-field

Conversation

@edalzell

@edalzell edalzell commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Declare location as a group (name + nested coordinates) beside existing online_url
  • ICS reads location.name / location.coordinates / online_url only — drops address, link, URL sniff, and top-level coordinates
  • Replace protected icsAddress() / eventUrl() with icsLocation() / icsUrl()
  • Docs + mapping table for the 7.0 contract; start upgrade notes (migrator in Add update script migrating location fields to 7.0 #196)

Closes #195
Part of #190

Blocked for Prime sites until transformstudios/prime#834.

Test plan

  • vendor/bin/pest (142 passed)
  • CP: Location group + Online URL on an event blueprint importing events::event
  • Physical / online-only / hybrid ICS downloads match the mapping table
  • Confirm legacy address / link / string location no longer appear in ICS

@edalzell
edalzell merged commit 28544da into main Sep 9, 2026
25 checks passed
@edalzell
edalzell deleted the feature/location-field branch September 9, 2026 22:35
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.

Add nested location group; remove address, link, and sniffing

1 participant