Support timezone for logged in user #93

Merged
merged 1 commit into from Sep 2, 2011

Conversation

Projects
None yet
2 participants
Contributor

keedi commented Aug 30, 2011

http://mojomojo.2358427.n2.nabble.com/Setting-time-zone-td6001153.html

If timezone support is already implemented, please discard this message.

In database schema, person table has a timezone column.
So, I use that column and add timezone form, tz method.
After that I convert DateTime objects which are in the view using c.tz() method.
atom.tt is already specified "-00:00", so I didn't touch them.
Only if user is logged in, then timezone will be applied. :-)

P.S.
I'm not good at Catalyst and don't know Catalyst best practice.
So, I do not know whether adding tz() method in MojoMojo.pm is appropriate.

@marcusramberg marcusramberg added a commit that referenced this pull request Sep 2, 2011

@marcusramberg marcusramberg Merge pull request #93 from keedi/master
Support timezone for logged in user
5e4d7ee

@marcusramberg marcusramberg merged commit 5e4d7ee into mojomojo:master Sep 2, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment