Skip to content

Conversation

kangkyu
Copy link
Contributor

@kangkyu kangkyu commented May 3, 2018

in the Facebook response

created_time = "2018-04-13T15:00:01+0000" 

but current way with strptime it parses into a local time, that is not exact

require 'time'
# => true
Time.strptime(created_time,  '%Y-%m-%dT%H:%M:%S+0000')
# => 2018-04-13 15:00:01 -0700
Time.parse created_time
# => 2018-04-13 15:00:01 +0000

"2018-04-13T15:00:01+0000" in the Facebook response
Time.strptime 2018-04-13 15:00:01 -0700
Time.parse 2018-04-13 15:00:01 +0000
@coveralls
Copy link

coveralls commented May 3, 2018

Coverage Status

Coverage increased (+0.007%) to 97.989% when pulling 0bd453c on parse-time into 9f1c450 on master.

@kangkyu kangkyu merged commit 7398a8e into master May 5, 2018
@kangkyu kangkyu deleted the parse-time branch May 5, 2018 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants