Skip to content
This repository has been archived by the owner on Dec 25, 2024. It is now read-only.

Money Poolの削除方法を論理削除に変更 #48

Merged
merged 2 commits into from
Nov 6, 2023
Merged

Money Poolの削除方法を論理削除に変更 #48

merged 2 commits into from
Nov 6, 2023

Conversation

walnuts1018
Copy link
Owner

@walnuts1018 walnuts1018 commented Nov 6, 2023 •
edited
Loading

Why

  • 外部キー制約？のせいでPaymentがあると消せない
  • MoneyPoolを消すと戻せないの良くないかなぁどうかなぁの舞

@walnuts1018 walnuts1018 linked an issue Nov 6, 2023 that may be closed by this pull request
外部キー制約 #42
Closed
@walnuts1018 walnuts1018 changed the title Add is_deleted column to money_pool table and Money Poolの削除方法を論理削除に変更 Nov 6, 2023
@walnuts1018 walnuts1018 closed this Nov 6, 2023
@walnuts1018 walnuts1018 reopened this Nov 6, 2023
@walnuts1018 walnuts1018 removed the request for review from wolfmagnate November 6, 2023 09:24
@walnuts1018 walnuts1018 marked this pull request as draft November 6, 2023 09:24
@walnuts1018
Add deleted_at field to money_pool table and
fc2f7ee 
update DeleteMoneyPool method
@walnuts1018 walnuts1018 marked this pull request as ready for review November 6, 2023 09:31
@walnuts1018 walnuts1018 merged commit ce03c6f into main Nov 6, 2023
4 checks passed
@walnuts1018 walnuts1018 deleted the change-delete-type branch November 6, 2023 09:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

外部キー制約
1 participant
@walnuts1018