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

同じリアクション削除を同時に行うとリアクションカウントがおかしくなることがあるのを修正 #6253

Merged
merged 3 commits into from Apr 13, 2020
Merged

同じリアクション削除を同時に行うとリアクションカウントがおかしくなることがあるのを修正 #6253

merged 3 commits into from Apr 13, 2020

Conversation

mei23
Copy link
Contributor

@mei23 mei23 commented Apr 13, 2020 •
edited

Summary

Fix #6252
Fix #6224

リアクションを連打すると他人のリアクションが消える (というかカウントがおかしくなる) ことがあるのを修正

@mei23 mei23 mentioned this pull request Apr 13, 2020
Closed
@mei23 mei23 changed the title リアクションを連打すると他人のリアクションが消えることがあるのを修正 Fix #6252 同じリアクション削除を同時に行うとリアクションカウントがおかしくなることがあるのを修正 Apr 13, 2020
@syuilo
Copy link
Member

syuilo commented Apr 13, 2020

TypeORMはPostgreSQLの affected サポート無かった気がします

@mei23
Copy link
Contributor Author

mei23 commented Apr 13, 2020 •
edited

とりあえずちゃんと 0 とか 1 返ってきてるみたい

@syuilo syuilo merged commit 916512f into misskey-dev:develop Apr 13, 2020
@syuilo
Copy link
Member

syuilo commented Apr 13, 2020

ありがとうございます🥰🥰🥰

@mei23
Copy link
Contributor Author

mei23 commented Apr 13, 2020

いちおうコード的にも両方対応してるみたいです
DELETE typeorm/typeorm@d7ec7f0
UPDATE typeorm/typeorm#4432

typeorm/typeorm#2415
のIssueは、最初postgresの話してて、postgresの問題はなおってるのにcommitだけでリンクとかされてないからそこだけ見ててもわからない感じ。

@mei23 mei23 deleted the mei-a0413-6252 branch April 13, 2020 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@mei23 @syuilo