Skip to content
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

Config/migrationfile #10

Merged
merged 2 commits into from Mar 12, 2024
Merged

Config/migrationfile #10

merged 2 commits into from Mar 12, 2024

Conversation

komody
Copy link
Owner

@komody komody commented Mar 12, 2024

やったこと

  • 全てのforeignIdにcascadeOnUpdate()とcascadeOnDelete()を追加
  • 2024_02_29_235949_change_like_count_default_value_in_posts_table.phpの内容を2024_02_14_080733_create_posts_table.phpに統合

@inoue0124
Copy link
Collaborator

手元でmigration:refreshしてみて問題なければ大丈夫だと思います〜

@komody
Copy link
Owner Author

komody commented Mar 12, 2024

コメントありがとうございます。
スクリーンショット 2024-03-13 0 10 30
問題なくrefreshできましたのでマージします。

@komody komody merged commit 5368305 into main Mar 12, 2024
@komody komody deleted the config/migrationfile branch March 12, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inoue0124 inoue0124 inoue0124 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@komody @inoue0124