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 list ical #133
Fix list ical #133
Conversation
… one is specified
In iCal timezone names are most often of the type EDT or CEST or IST, but IST can be applied to the zones: ['Asia/Colombo', 'Asia/Kolkata', 'Europe/Dublin'] So we need to keep a mapping of the localized timezone name and their UTC offset to be able to track down which is which. We also define some standard timezone acronym (EDT, CET, CEST) for which we can directly substitute the name for one pytz and datetime know. This should allow us to load our own iCal files.
Some background information about the last three commits: http://blog.pingoured.fr/index.php?post/2014/07/25/The-Joy-of-timezones |
in output.data) | ||
self.assertTrue('li class="message">Calendar uploaded</li>' | ||
in output.data) | ||
#if 'JENKINS' not in os.environ: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why run this in jenkins now? Why was it not run in jenkins before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tbh, I wonder if it was failing and I am not even sure it was run at all.
I have no idea why it was there (I was surprised to see this there), I should git blame for this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit that introduced the change is: 821781e the commit message is nicely helpful but looks like the problem is not occurring anymore locally
One question, but it looks good to me. Nice work. :D |
Cool, 👍 |
No description provided.