Fix: fetch events for bots

[AdityaJ2305]

Proposed Changes

Fix: #554

Reviewer(s)

@rithviknishad

[netlify]

👷 Deploy request for leaderboard-develop pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	1948463

[rithviknishad]

If this change prevented the issue, the previous change also prevented the issue... Are you sure this is right?

[AdityaJ2305]

If this change prevented the issue, the previous change also prevented the issue... Are you sure this is right?

Let me check 😅

[AdityaJ2305]

Hey @rithviknishad, I don’t think this issue is coming from this piece of code. There seems to be something else causing it.

[rithviknishad]

You're right. But from where else it should come? 🤔

[rithviknishad]

Looking at this: https://flatgithub.com/ohcnetwork/leaderboard-data?filename=data%2Fgithub%2Fdependabot%5Bbot%5D.json&sha=98a8603b63dd721a726a2e52c14014067c7e1f0e&tab=activity

Looks like only PR Merged events are there. And I had merged dependabot's PRs that day. So even though I was the one who merged it, the author of the PR is still dependabot. So likely some incorrect logic there maybe?

The Author of the PR should get the event and not the one who merged it.

leaderboard/scraper/src/github-scraper/parseEvents.ts

Lines 204 to 211 in be46e0b

	appendEvent(event.payload.pull_request.user.login, {
	type: "pr_merged",
	title: `${event.repo.name}#${event.payload.pull_request.number}`,
	time: eventTime?.toISOString(),
	link: event.payload.pull_request.html_url,
	text: event.payload.pull_request.title,
	turnaround_time: turnaroundTime,
	});

[AdityaJ2305]

Looking at this: https://flatgithub.com/ohcnetwork/leaderboard-data?filename=data%2Fgithub%2Fdependabot%5Bbot%5D.json&sha=98a8603b63dd721a726a2e52c14014067c7e1f0e&tab=activity

Looks like only PR Merged events are there. And I had merged dependabot's PRs that day. So even though I was the one who merged it, the author of the PR is still dependabot. So likely some incorrect logic there maybe?

The Author of the PR should get the event and not the one who merged it.

leaderboard/scraper/src/github-scraper/parseEvents.ts

Lines 204 to 211 in be46e0b

	appendEvent(event.payload.pull_request.user.login, {
	type: "pr_merged",
	title: `${event.repo.name}#${event.payload.pull_request.number}`,
	time: eventTime?.toISOString(),
	link: event.payload.pull_request.html_url,
	text: event.payload.pull_request.title,
	turnaround_time: turnaroundTime,
	});

Let me check

@rithviknishad, Am I understanding it wrong? It is okay na if the author is dependabot for its PRs.
The error is why it is not getting ignore due to this line 👇

leaderboard/scraper/src/github-scraper/parseEvents.ts

Line 150 in be46e0b

if (isBlacklisted(user)) continue;

[rithviknishad]

inactive