Skip to content

[WIP]/events の追加 #270

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 26 commits into from
Closed

[WIP]/events の追加 #270

wants to merge 26 commits into from

Conversation

AnaTofuZ
Copy link
Member

@AnaTofuZ AnaTofuZ commented Feb 28, 2018 •
edited
Loading

該当issue

#258

やること

  • /events へのルーティング
  • テストコード(画面表示)
  • コントローラー、viewの用意
  • 具体的に今後のDojoのみを表示するロジックの実装
  • テストコードの記載

困っていること

  • 具体的にどうすれば今後のイベントが取得できるか

@AnaTofuZ
add notes order
1c5b3cb 
Fix coderdojo-japan#268

> 今回のマージでorder 周りの設定手順が変わったと思うので、docs/how-to-add-dojo.md のドキュメントにも反映しておくと良いかも
> cf. coderdojo-japan#267

`order`はnameから取っている為、市町村と同じDojo名だと省略可能な旨を追記
した
@AnaTofuZ
modofied orde notes
b75746d 
脚注がGithubだと展開されない為別途記述した
@AnaTofuZ
update_date_dojo.md
9c1212a
@AnaTofuZ
touch events.html.haml and add static_pages_controller_spec test
c02f95c 
`/events`を作りたい為、雛形hamlファイルとコントローラー
及び適切に`/events` が表示できるかのテストコードを追加した。
@AnaTofuZ AnaTofuZ changed the title /events の追加 [WIP]/events の追加 Feb 28, 2018
@AnaTofuZ
Copy link
Member Author

本番のDBでもEventHistory.whereして取れなかったが、これは統計情報の為の集約の状況でログを取っているからであった。別途APIにアクセスする必要がありそう

@AnaTofuZ
Copy link
Member Author

過去issue でAPI周りについて言及されているのは #12 なのでこのissue付近を読んでいく

@AnaTofuZ
Copy link
Member Author

AnaTofuZ commented Feb 28, 2018 •
edited
Loading

ローカルのDBをherokuのpostgresqlと同期したところ
EventHistory.where(evented_at:Time.zone.now..Time.zone.now.end_of_year) で塩尻のデータが取得できた

@AnaTofuZ
Copy link
Member Author

取得できたのは統計情報のrakeを今週として実行したからであった。
統計のrakeタスクを今月までで実行すると塩尻のデータが入手できるが、開催済みのイベントとしてカウントされてしまう。
DBに保存せずAPIを叩けば塩尻のデータが入手出来ると思う。

@AnaTofuZ
Copy link
Member Author

AnaTofuZ commented Mar 1, 2018

statistics からAPI関連の実装を抜き出す為に、内部構造を調査中

yasulab and others added 20 commits March 1, 2018 21:52
@yasulab
Add IslayTouch to 水戸/ひたちなか
8880b62
@yasulab
Update order column in yaml maigration task
e29b99f
@yasulab
Tweak layout of code in yaml migration
c214365
@yasulab
Change order of columns due to updating yaml migration task
cea0909
@yasulab
Merge pull request coderdojo-japan#273 from coderdojo-japan/add-order…
24da0f0 
…-to-yaml

Update order column with new yaml migration task
@yasulab
🆕 Add CoderDojo 津山
e14ede1
@yasulab
Merge pull request coderdojo-japan#274 from coderdojo-japan/add-tsuya…
5fbe49f 
…ma-dojo

🆕 Add CoderDojo 津山
@yasulab
Add static number of dojos to flush meta tags in DB
226f174
@yasulab
Revert to show number of dojos dynamically
b23815b
@yasulab
Merge pull request coderdojo-japan#269 from AnaTofuZ/modified_how-to-…
db6d12b 
…add-dojo.md

add notes `order`
@yasulab
Add new lines and shorten link to README
fa396c1
@yasulab
Update how to add dojo document
e2f3d32
@yasulab
Add example of adding dojo
8ba4ed8
@yasulab
Add Japanese message when failed to detect city code
5f88a07
@yasulab
Add CoderDojo 甲斐竜王
477f973
@AnaTofuZ
modofied namespace statiscs/providers/ to providers/
2a0529a 
statiscs/providersにはDojoのイベントAPIにアクセスする為の実装がある.
直近のイベント情報を取得する際も、APIアクセスが必要となる為この資産を使いたい。

> statistics 系の資源を使いつつ、直近情報を入手する実装がやっぱり良さそう
> ネームスペースが気持ち悪いと思うので、Statisticsの外に出してしまうと良いかもしれないですねぇ。

そこで名前空間をstatiscs/providersからprovidersに変更した
@AnaTofuZ
remove module Statistics namespace for lib/providers/*.rb
1a03b3c 
先程のcommitでnamespaceの変更を行った為、Statisticsの名前空間を削除した
@AnaTofuZ
Merge branch 'add_events_pages' of ssh://github.com/AnaTofuZ/coderdoj…
cb0a915 
…o.jp into add_events_pages
@AnaTofuZ
lib/statistics.rbでの providers の名前空間からstatisticsを外した
4b8f227 
`providers` の名前空間を`statistics` から外した為、require_relativeのパ
スを修正した
@AnaTofuZ
名前空間の変更に伴うlib/tasks/statistics.rakeでのStatistics::Providersの名前修正
ec1f07b 
Statistics::ProvidersからProvides::に名前空間を修正した為
使用しているrakeファイルの名前空間を修正した
AnaTofuZ added 2 commits March 5, 2018 11:19
@AnaTofuZ
git mv spec/lib/statistics/providers -> spec/lib/providers/*
1e412e4
@AnaTofuZ
spec/lib/providers* でのテスト対象の名前空間の修正
4721898 
テストコードが`Statistics::Providers` を参照していた為、`Statistics` 名
前空間を外した
@AnaTofuZ
Copy link
Member Author

AnaTofuZ commented Mar 5, 2018

一旦名前空間の切り出しとしてPRを作る方が粒度的に良いと思われるのでcloseします

@AnaTofuZ AnaTofuZ closed this Mar 5, 2018
@AnaTofuZ AnaTofuZ deleted the add_events_pages branch March 5, 2018 02:28
This was referenced Mar 5, 2018
Merged
Closed
@AnaTofuZ AnaTofuZ mentioned this pull request Mar 16, 2018
Merged
3 tasks
AnaTofuZ added a commit that referenced this pull request Mar 19, 2018
@AnaTofuZ
upcoming_eventsテーブルを定義した
8e716cb 
 #270 からの派生です。
直近のイベント情報を保存させたい為にupcoming_eventsとしてテーブルを定義
しました。
またdojo_idを持たせない等の正規化も行っています
@aokabin aokabin mentioned this pull request Apr 13, 2018
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AnaTofuZ @yasulab