Use event timezone if no timezone specified in get param of ical feed (especially for Agenda feed)

The issue is, for the ics feed, if the get parameter time_zone is not specified in the url, it causes a rails error.

The solution is to default to the event’s timezone when the get param is not specified to avoid the error condition.

1 Like

Also reported here: ics requires a parameter · Issue #44 · paviliondev/discourse-events · GitHub

Hi @angus, I thought about this. The issue is that there are multiple events in a feed and different timezones could be used for each event. I think we should default to UTC. Thoughts?

@fzngagan How about we default to the default timezone site setting if it’s filled, then UTC if it’s not?

1 Like

Ok, I’ve made a PR for this.

1 Like

@fzngagan Merged :+1:

1 Like

This topic was automatically closed after 4 days. New replies are no longer allowed.