Skip to content

2018年の統計情報を追加 #376

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

Merged
merged 3 commits into from
Apr 1, 2019
Merged

2018年の統計情報を追加 #376

merged 3 commits into from
Apr 1, 2019

Conversation

odlovesq
Copy link
Contributor

@odlovesq odlovesq commented Jan 22, 2019

背景

CoderDojoでは統計情報ページで1年単位のグラフをまとめている。
年が明けて2019年になったので、2018年の統計情報も追加したい。

やること

cf. #370 (comment)

関連issue

#370

@odlovesq odlovesq self-assigned this Jan 22, 2019
年も明けて2019年になったので、統計情報ページに2018年分の統計情報も公開したいため
@odlovesq
Copy link
Contributor Author

同ページ内の最新データ項目で、「2018年のデータも含めた最新の統計情報は次のとおりです。 」という文言が静的に記述されています。
アクティブな道場の数を都度DBから取得しているようなので、文言も「2019年」に修正したほうがいいかと思ったのですが、こちらも一緒に対応してもよろしいでしょうか?
default

@yasulab
Copy link
Member

yasulab commented Jan 24, 2019

初 PR おめでとうございます! 😆🎉✨
Description も分かりやすくていいですね! 😳

アクティブな道場の数を都度DBから取得しているようなので、文言も「2019年」に修正したほうがいいかと思ったのですが、こちらも一緒に対応してもよろしいでしょうか?

はい!こちらも対応していただけると助かります (>人< )✨

現在 WIP となっていますが、 何が完了したら WIP が解除されるのか 判断しづらかったため、僕の方で Description を一部更新しておきました ;)

具体的には、次のようにしておくとなお分かりやすいですね! 今後の PR のご参考になれば 😆

  • 「やること」のチェックリストに未達成の項目がある => WIP 設定
  • 「やること」のチェックリストがすべて完了している => WIP 解除

やること

cf. #370 (comment)

@yasulab yasulab changed the title [WIP]2018年の統計情報を追加 [WIP] 2018年の統計情報を追加 Jan 24, 2019
アクティブな道場の数を都度DBから取得していて、常に最新のデータが表示されているので更新しました。
@odlovesq
Copy link
Contributor Author

Descriptionの更新ありがとうございます!!
次から参考にさせていただきます!!!

アクティブな道場の数を都度DBから取得しているようなので、文言も「2019年」に修正したほうがいいかと思ったのですが、こちらも一緒に対応してもよろしいでしょうか?

はい!こちらも対応していただけると助かります (>人< )✨

こちらも対応しました

@chicaco
Copy link
Contributor

chicaco commented Mar 5, 2019

たいへんお待たせしてしまいましたが、 #371 + #375 が無事解決しました。
準備が整ったらマージしてください!

@yasulab yasulab changed the title [WIP] 2018年の統計情報を追加 2018年の統計情報を追加 Mar 6, 2019
@yasulab
Copy link
Member

yasulab commented Apr 1, 2019

こちら問題なさそうなのでマージしますね ;) cc/ @odlovesq

@yasulab yasulab merged commit 97b09bd into master Apr 1, 2019
@yasulab yasulab deleted the add_2018_stats_data branch April 1, 2019 01:30
@yasulab
Copy link
Member

yasulab commented Apr 1, 2019

デプロイ後、僕の方でもデータをチェックして、何か不具合などあれば僕の方で対応しますね! ✅ 🔍 👀 💨
https://coderdojo.jp/stats

@odlovesq
Copy link
Contributor Author

odlovesq commented Apr 2, 2019

マージありがとうございます!

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.

3 participants