Skip to content
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

Add handling of JsonToken.Date #65

Merged
merged 1 commit into from Jul 3, 2014
Merged

Conversation

@a4lg
Copy link
Contributor

a4lg commented Jul 3, 2014

Issue #64 の修正です。CoreTweet 内の DateTimeOffsetConverter では JsonToken.Date という日時を示すトークンの処理が抜けていたため、この形で処理が可能だった ISO 8601 拡張フォーマットのトークン (JsonToken.Date) がうまく解釈できないということが根本的な原因でした。
本コミットにおいては JsonToken.Date の処理を行うことで Issue #64 における Trends API 関連のタイムスタンプを正しく処理できるようにしました。本トークンの Value は DateTimeOffset または DateTime ですが、後者の場合 UTC に変換後 UTC の DateTimeOffset に変換するということを行っており、ラウンドトリップ性が失われています――が、さしたる問題は出ないでしょう。

@a4lg
Copy link
Contributor Author

a4lg commented Jul 3, 2014

あれ、Travis CI テストが失敗……って、Ruby プログラム前提の CI が走っている? 修正としては問題ないはずです。

@azyobuzin
Copy link
Member

azyobuzin commented Jul 3, 2014

CI はまだ設定してないので必ず失敗します

azyobuzin added a commit that referenced this pull request Jul 3, 2014
Add handling of JsonToken.Date
@azyobuzin azyobuzin merged commit 598a4c3 into CoreTweet:master Jul 3, 2014
1 check failed
1 check failed
continuous-integration/travis-ci The Travis CI build failed
Details
@a4lg a4lg deleted the a4lg:json-date-fix branch Jul 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.