-
Notifications
You must be signed in to change notification settings - Fork 75
スキーマファイルから不要な行を削除 #9482
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
base: main
Are you sure you want to change the base?
スキーマファイルから不要な行を削除 #9482
Conversation
📝 WalkthroughWalkthroughデータベーススキーマから Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related issues
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@kitarou888 |
|
@ryufuta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ryufuta
確認できましたのでApproveとさせていただきます!
自分のPRを今一度確認しましたが、完全にマイグレーションをし忘れていました・・・
このたびはありがとうございました🙇♂️
|
@kitarou888 |
fcb9370 のコミットで反映し忘れた
data_migrationsテーブルはデータマイグレーション実行時に自動的に作成されるためスキーマファイルで管理する必要がない
464837f to
ec88916
Compare
|
@okuramasafumi |
Issue
概要
スキーマファイル(
db/schema.rb)の2箇所の不備を修正してマイグレーションファイルとの不整合を解消した。詳細はIssueを参照。
補足
既存のプロダクションコードとテストコードに
usersテーブルのjob_seekingカラムに依存した箇所はなかった。リポジトリ内に
job_seekingという変数は以下の2つに関連したものとして見つかるが、元々存在していたusersテーブルのjob_seekingカラム(卒業生が就職活動中かどうかを表すブール値)に依存したものではない。usersテーブルのcareer_pathカラムのenum値としてのjob_seekingjob_seeking(管理画面のユーザー一覧を就職希望と決済方法でも絞り込めるようにした #8737 で追加された機能。usersテーブルのjob_seekerカラムに依存)変更確認方法
UI、振る舞い、ロジックの変更はない。
Issueに記載されている不要な行(マイグレーションをやり直すと削除される行)が
db/schema.rbから削除されていることをFiles changedを見て確認する。Summary by CodeRabbit
✏️ Tip: この高レベル概要はレビュー設定でカスタマイズできます。