Add migration files

[hayato24s]

db/migrations

既存のマイクロサービスを立ち上げた後、pgAdminからDDLをコピペしてマイグレーションファイルを作成しました。

db/models

DBのテーブル構造からGoのコードを生成するsqlboilerというORMを使用しています。
自動生成です。

schemaspy

schemaspyというツールで自動生成してます。

レビューして頂きたい点

マイグレーションファイルのクエリが既存のサービスのDB構造と矛盾していないか確認して頂きたいです。
pgAdminからコピペしてきたのですが、大丈夫でしたでしょうか？

[hayato24s]

下記のスクリーンショットは既存のマイクロサービスを立ち上げてpgAdmin4からDDLを確認しているところです。
このDDLをコピペしてtwinte-backのマイグレーションファイルを作成しました。

僕としては、twinte-backでも同様にサービスを起動してからpgAdmin4でDB構造を表示し、正しそうか目視で判断すれば良いかと考えています。

twinte-backでDB構造を確認するための手順

1. make up
2. make bash
3. make migrate-up
4. pgAdmin4でDB構造を目視で確認する

既存のマイクロサービスでのpgAdminにおけるスクリーンショット

[takonasu]

チェック量が多いので時間がかかっていますがチェック中なのでもう少々お待ちを～

[takonasu]

差分をチェックした項目

tables

• already_reads
• course_methods
• course_recommended_grades
• course_schedules
• courses
• payment_users
• registered_course_tags
• registered_courses
• schema_migrations
• session
• tags
• ✖ user_authentications
  差分画像（左が今回のPRのもの、右がuser-service）
  
• ✖ users
  差分画像（左が今回のPRのもの、右がuser-service）
  

sequences

• ✖ course_methods_id_seq
  OWNED BY course_methods.id欠損
• ✖ course_recommended_grades_id_seq
  OWNED BY course_recommended_grades.id欠損
• ✖ course_schedules_id_seq
  OWNED BY course_schedules.id欠損
• user_authentications_id_seq

types

• course_methods_method_enum
• course_schedules_day_enum
• course_schedules_module_enum
• registered_courses_methods_enum
• user_authentications_provider_enum

[hayato24s]

@takonasu

schemaspyを設定するの忘れていました。すみません🙇‍♂️

リポジトリをローカルにcloneした後、ブラウザからこちらのファイルを表示すると、下記のような画面が見れると思います。

こちらの画面からDBのテーブル構造を確認していただけるとありがたいです。

ちなみに、DBのテーブル構造からschemaspyのhtmlなどを生成するにはmake schemaspyを実行すればできます。

[hayato24s]

user_authentications.social_idとpayment_users.idが関連しちゃっています。

pgAdmin4では特にforeign key制約などは存在しません。

なんでだろう？バグかな？

[hayato24s]

recommended_gradesなどのcourse_idがnullableなのはなぜ？

[takonasu]

@hayato24s ファイルの差分を確認しました。
差分が出ている個所（Nullable）等確認してください。
既存のDBスキーマでの疑問点は、設計は @SIY1121 が担当したので、聞いてもらえると解決できるかと！

[hayato24s]

@takonasu

user_authenticationsとusersに関しては左の今回のPRのものが正しいです。
実際に、本番環境のDBでは左のようになっていると思われます。
ですので、本PRで修正をする必要はないと思われます。

• ✖ user_authentications
  差分画像（左が今回のPRのもの、右がuser-service）
  
• ✖ users
  差分画像（左が今回のPRのもの、右がuser-service）
  

[hayato24s]

@takonasu

seqの箇所は修正しました。

sequences

• ✖ course_methods_id_seq
  OWNED BY course_methods.id欠損
• ✖ course_recommended_grades_id_seq
  OWNED BY course_recommended_grades.id欠損
• ✖ course_schedules_id_seq
  OWNED BY course_schedules.id欠損

[takonasu]

ごめんなさい、確認してApproveするのを失念していましたmm