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

直近のイベント情報を格納するDBスキーマの定義 #287

Merged
merged 1 commit into from Mar 19, 2018

Conversation

Projects
None yet
3 participants
@AnaTofuZ
Member

AnaTofuZ commented Mar 16, 2018

背景

  • #270events を作るに辺り直近のイベント情報が欲しい
  • 現在のDB構成は過去のイベントログしか保存しない為、直近のイベントを保存させたい
  • その為不確定な参加者を除くイベントスキーマを作成したい
    関連PR: #277

やること

  • be rails g migration CreateFutureEvents dojo_event_service_id:integer event_id:string event_url:string evented_ad:datetime
  • 詳細なスキーマの変更
  • 命名規則の確認

今回は未来のイベントなのでFutureEventsとして定義しています

その後名前を Future から Upcoming に修正しました

@AnaTofuZ AnaTofuZ requested review from yasulab and nalabjp Mar 16, 2018

@yasulab yasulab changed the title from 直近のイベント情報を格納するDBschemeの定義 to 直近のイベント情報を格納するDBスキーマの定義 Mar 16, 2018

@yasulab

This comment has been minimized.

Member

yasulab commented Mar 16, 2018

今回は未来のイベントなのでFutureEventsとして定義しています

@AnaTofuZ 「未来のイベント」だと10年後とか20年後とかも意味合いとしては含まれてしまうから、今回の文脈では UpcomingEvents の方がしっくりくるかなぁ🤔💭

https://eow.alc.co.jp/search?q=upcoming

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 16, 2018

@yasulab なるほど Upcoming !! ありがとうございます。
branchの名前がfutureになっていますが Upcomingにmvして修正してみます

@AnaTofuZ AnaTofuZ changed the title from 直近のイベント情報を格納するDBスキーマの定義 to [WIP]直近のイベント情報を格納するDBスキーマの定義 Mar 16, 2018

@AnaTofuZ AnaTofuZ changed the title from [WIP]直近のイベント情報を格納するDBスキーマの定義 to 直近のイベント情報を格納するDBスキーマの定義 Mar 16, 2018

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 16, 2018

@yasulab 名前を Upcoming に変更してみました!

@yasulab

This comment has been minimized.

Member

yasulab commented Mar 16, 2018

@AnaTofuZ FutureEvents から UpcomingEvents に変更されたコミットのログって残しておく必要ありますかね...? 🤔 なければ Squash してしまった方が良さそうです >< 💦

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 16, 2018

確かに結果的にはいらないですね…!!Squashしてみます

@yasulab

This comment has been minimized.

Member

yasulab commented Mar 16, 2018

特に今回の場合、PR内でマイグレーションが完結しているので、Squash しちゃった方が後からコミットログを見る人に親切そうです 😊

db/schema.rb Outdated
@@ -61,6 +61,24 @@
t.index ["region"], name: "index_prefectures_on_region"
end
create_table "recently_events", force: :cascade do |t|

This comment has been minimized.

@nalabjp

nalabjp Mar 17, 2018

Contributor

rename前の定義が残っているのでここも削除しておいてほしいです😌

@nalabjp

rename前のテーブル定義の削除とコミットのsquashが入ればLGTMです 👍

@AnaTofuZ AnaTofuZ force-pushed the create_feature_events_table branch Mar 19, 2018

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 19, 2018

使っていないテーブルの削除と過去のcommitをSquash してみました

@yasulab

This comment has been minimized.

Member

yasulab commented Mar 19, 2018

@AnaTofuZ Typo 👀

upcoming_eventsテーブル をを 定義した

@yasulab

This comment has been minimized.

Member

yasulab commented Mar 19, 2018

上記の Typo 以外は問題なさそうです! Typo 修正したらマージしちゃいますね ;)

upcoming_eventsテーブルを定義した
 #270 からの派生です。
直近のイベント情報を保存させたい為にupcoming_eventsとしてテーブルを定義
しました。
またdojo_idを持たせない等の正規化も行っています

@AnaTofuZ AnaTofuZ force-pushed the create_feature_events_table branch to 8e716cb Mar 19, 2018

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 19, 2018

typo修正しました

@yasulab yasulab merged commit 29f24fb into master Mar 19, 2018

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@yasulab

This comment has been minimized.

Member

yasulab commented Mar 19, 2018

マージしました! 👍

@yasulab yasulab deleted the create_feature_events_table branch Mar 19, 2018

@AnaTofuZ

This comment has been minimized.

Member

AnaTofuZ commented Mar 19, 2018

ありがとうございます! 🙏

@yasulab yasulab referenced this pull request Jun 5, 2018

Merged

道場を非表示にする仕組みの追加 #340

5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment