-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Fix activity inconsistency when delete pull request #26285
Fix activity inconsistency when delete pull request #26285
Conversation
It's not right. I fix it by the way in Bypass MariaDB performance bug of the "IN" sub-query, fix incorrect IssueIndex #26279 Thank you for your finding and report 😆 |
You're right, I didn't notice the operation related to |
It's will be fixed in #26279 |
…tency-when-delete-pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review #26279 which has a full fix for this problem and has tests.
Description:
When I delete a PR, I find that the entry is still in the feed stream on the home page. If I click on it, I get 404 and 500 errors.
Causes:
When deleting the Issue, the corresponding
activity
was deleted usingissue.ID
by mistake, when it should have usedissue.Index
.Screenshot(gif):