Public user activity includes wrong PullRequestEvent when a PR gets closed #194341
Replies: 3 comments
-
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
Beta Was this translation helpful? Give feedback.
-
|
This seems to be the case for any PR that gets closed instead of merged? E.g., in event 8491690930 the created_at time matches the date of the last comment in that PR, but it was closed only several days later. |
Beta Was this translation helpful? Give feedback.
-
|
Merges are also misattributed to the user that created the PR instead of the maintainer that did the merge, see e.g. event ID 9252311762. I'm starting to think that PullRequestEvent entries are too broken to be a meaningful data source. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
🏷️ Discussion Type
Bug
💬 Feature/Topic Area
API
Body
Edit: updated title to reflect that this isn't limited to drafts
When a pull request that was previously moved back to draft gets closed, the actor's activity timeline obtained via
/users/USERNAME/events/publicwill include the entry for the draft change, but with a payload for the close action.Example excerpt:
Beta Was this translation helpful? Give feedback.
All reactions