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

最終更新日時を正規化する #53

Open
yumetodo opened this issue Dec 4, 2017 · 8 comments
Open

最終更新日時を正規化する #53

yumetodo opened this issue Dec 4, 2017 · 8 comments

Comments

@yumetodo
Copy link
Member

yumetodo commented Dec 4, 2017

最終更新日時として表示されるタイムスタンプがUTCなのはまあいいとして、

最終更新日時: 2017年12月04日 02時05分37秒

ではUTCなのかJSTなのかそれ以外なのかわからない。

最終更新日時: 2017年12月04日 02時05分37秒 (UTC)

とするなり、locale見て時差考慮するなりするべきではないか

@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

ああこれは https://github.com/github/time-elements で改善しようかとちょっと考えてました。直したいですね。静的HTMLの吐き方とJSロード時のタイムゾーン対応動的書き換え部分はGitHubのコメント日時とかに準じるという方針でとりあえず考えてます(いまのところ喫緊で作業する予定はないです)

@saki7 saki7 changed the title 最終更新日時として表示されるタイムスタンプがUTCなのにその旨の記述がない 最終更新日時を正規化する Dec 4, 2017
@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

UTC表記を直すよりも、もっと包括的に直した方がまとめて解決すると思うので、Issueのタイトルを変えました。これが不正確な場合は、変更していただければと思います。

@yumetodo
Copy link
Member Author

yumetodo commented Dec 4, 2017

@saki7 少なくとも正規化をどう実装していつやるかはさておき、現状みてあれがUTCと分かる人はいないんじゃないかと思っているんですが、その点(UTC)を追記するという軽微な変更(軽微ですよね?)をさしあたり入れるべきではないでしょうか?(時間がなくてまだどこのコードがここを吐いているか調べていない)

@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

その趣旨の変更であれば、PR頂ければとりあえずマージはします。

@saki7
Copy link
Contributor

saki7 commented Dec 13, 2017

@yumetodo ちなみに

最終更新日時:
<span itemprop="datePublished" content="{{ latest_commit_info['last_updated'].strftime('%Y-%m-%dT%H:%M:%S'.encode('utf-8')).decode('utf-8') }}">
{{ latest_commit_info['last_updated'].strftime('%Y年%m月%d日 %H時%M分%S秒'.encode('utf-8')).decode('utf-8') }}
</span>

site_generator/run.py

Lines 426 to 434 in 8cc0359

def get_latest_commit_info(path):
commit_log = subprocess.check_output(['git', 'log', '-1', '--date=iso', '--pretty=format:%at %an', path + '.md'], cwd=settings.INPUT_DIR)
if not commit_log:
return None
timestamp, author = unicode(commit_log, encoding='utf-8').split(' ', 1)
return {
'last_updated': datetime.fromtimestamp(int(timestamp)),
'last_author': author,
}

@yumetodo
Copy link
Member Author

@saki7 あぶない、自分で立てておいてすっかり忘れていました。とりかかります。

@saki7 saki7 closed this as completed in 7c7844f Dec 14, 2017
saki7 added a commit that referenced this issue Dec 14, 2017
@saki7 saki7 reopened this Dec 14, 2017
@saki7
Copy link
Contributor

saki7 commented Dec 14, 2017

正規化は終わってないのでreopenしておきます

@yumetodo
Copy link
Member Author

あ、commitメッセージにfixとか書くとcloseされるの忘れてました・・・

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

No branches or pull requests

2 participants