Prefectures #175
Merged
Prefectures #175
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
おー、早速のPRが...!! ありがとうございます😸 LGTM なので、マージしますね! 🔧 💨 |
nalabjp
commented
Nov 3, 2017
| @@ -0,0 +1,5 @@ | |||
| class AddPrefectureIdToDojo < ActiveRecord::Migration[5.1] | |||
| def change | |||
| add_column :dojos, :prefecture_id, :integer | |||
There was a problem hiding this comment.
after: :idとか書きたいところなんですが、postgresqlはサポートしていないので書いていません。（多分、開発時に使っているsqlite3も）
cf: http://wiki.postgresql.org/wiki/Alter_column_position
あと、null: falseも指定したいところなのですが、Rake taskを使った既存レコードのmigrate時に落ちてしまうので、NOT NULL制約も外しました。
|
コメントしている間にマージされていた、早い🤣 |
|
何か破壊的な変更があったわけではなく、特に気になる点も見当たらなかったので早速マージさせていただきました! 😸 |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#174
地域（Region）を全国のdojo一覧で使用したいので、都道府県テーブル（prefectures）を追加して
Dojo belongs_to Prefectureとしました。ActiveHashを使うという手もあると思うんですが、
後々イベント履歴の統計情報をSQLで抽出することを考えると、
テーブルとして持っておいたほうが良さそうだったのでマスターテーブルにしました。