{"action":"created","issue":{"url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/5","labels_url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/5/comments","events_url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/5/events","html_url":"https://github.com/petropavel13/pg_rrule/issues/5","id":112648265,"number":5,"title":"The timezone flavor of get_occurrences() fails to find an icaltimezone, causing occurrences to be offset when transitioning in to/out of daylight savings time.","user":{"login":"groner","id":47317,"avatar_url":"https://avatars.githubusercontent.com/u/47317?v=3","gravatar_id":"","url":"https://api.github.com/users/groner","html_url":"https://github.com/groner","followers_url":"https://api.github.com/users/groner/followers","following_url":"https://api.github.com/users/groner/following{/other_user}","gists_url":"https://api.github.com/users/groner/gists{/gist_id}","starred_url":"https://api.github.com/users/groner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/groner/subscriptions","organizations_url":"https://api.github.com/users/groner/orgs","repos_url":"https://api.github.com/users/groner/repos","events_url":"https://api.github.com/users/groner/events{/privacy}","received_events_url":"https://api.github.com/users/groner/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-10-21T18:13:47Z","updated_at":"2015-10-22T09:44:25Z","closed_at":null,"body":"If I'm not misunderstanding this, in order for libical to do the right thing it needs to have a timezone definition that includes daylight savings time.\r\n\r\nCurrently we're using the session timezone.  This seems like a reasonable default, as long as the timezone name postgres has is known by libical.  In a recurring calendar component, the timezone to use would be specified as a TZID parameter on the DTSTART property.  PostgreSQL's `timestamp with time zone` type only stores the timezone offset, so we need to provide the TZID some other way.\r\n\r\n@petropavel13 If this all makes sense to you, what do you think of adding variants of `get_occurrences()` that accept tzid as a parameter?\r\n"},"comment":{"url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/comments/150163494","html_url":"https://github.com/petropavel13/pg_rrule/issues/5#issuecomment-150163494","issue_url":"https://api.github.com/repos/petropavel13/pg_rrule/issues/5","id":150163494,"user":{"login":"petropavel13","id":6436245,"avatar_url":"https://avatars.githubusercontent.com/u/6436245?v=3","gravatar_id":"","url":"https://api.github.com/users/petropavel13","html_url":"https://github.com/petropavel13","followers_url":"https://api.github.com/users/petropavel13/followers","following_url":"https://api.github.com/users/petropavel13/following{/other_user}","gists_url":"https://api.github.com/users/petropavel13/gists{/gist_id}","starred_url":"https://api.github.com/users/petropavel13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petropavel13/subscriptions","organizations_url":"https://api.github.com/users/petropavel13/orgs","repos_url":"https://api.github.com/users/petropavel13/repos","events_url":"https://api.github.com/users/petropavel13/events{/privacy}","received_events_url":"https://api.github.com/users/petropavel13/received_events","type":"User","site_admin":false},"created_at":"2015-10-22T09:44:25Z","updated_at":"2015-10-22T09:44:25Z","body":"> If this all makes sense to you, what do you think of adding variants of get_occurrences() that accept tzid as a parameter?\r\n\r\nYeah. Sounds great. Waiting for PR ;)"}}