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.

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

[移行]DatabasesTruncate処理を修正 #1852

Merged
merged 2 commits into from Nov 14, 2023

Conversation

horitada
Copy link
Contributor

@horitada horitada commented Oct 16, 2023

DatabasesTruncate処理を修正

概要

データベース機能追加による移行不具合修正

レビュー完了希望日

不具合対応なので急ぎたいです

DB変更の有無

無し

チェックリスト

DatabasesTruncate処理を修正
@horitada horitada added bug バグ・不具合連絡 migration 移行 labels Oct 16, 2023
@horitada horitada changed the title DatabasesTruncate処理を修正 [移行]DatabasesTruncate処理を修正 Oct 16, 2023
@akagane99
Copy link
Contributor

うーむ、データベース機能追加してないから、修正内容があってるのかレビューできないなぁ。
牟田口はお手上げです。

【修正内容】

Databases::truncate();
↓
Databases::query()->delete();

やってることは変わってなさそうに、コードからは見えるんだけどなぁ。
DBテーブルに外部キー制約でも突っ込まれたかな?あてずっぽう。

データベース機能追加ってこれかな?

データベース機能追加した人に聞いてるみのが良いかと。
あとエラー内容も含めたら、聞かれる人も回答しやすいかと。

データベース外部キー参照解除に修正
@horitada
Copy link
Contributor Author

機能追加で外部キー参照していたので、外部キー解除処理に変更しました。

Copy link
Contributor

@akagane99 akagane99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

レビューオッケーです

@horitada horitada merged commit 0d20082 into master Nov 14, 2023
1 check passed
@horitada horitada deleted the migration_databases_truncate_error branch November 14, 2023 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグ・不具合連絡 migration 移行
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants