Add support for tracking commits pushed to branch (core) #601

rajku-dev · 2025-01-09T08:11:17Z

Part of #373

enable pushed_commits tracking for core team members with direct push access to org
enable pushed_commits tracking for non-core team contributors through forked repositories (Add support for tracking commits pushed to branch (Forked) #602)

* Configurable favicon.ico and logo.webp * modified link * fix: side navbar collapse on link click * remove my comments made my me * "fix:Scroll in Activity of user "

* hidden leaderboard filters in mobile view * Added filter icon button

* initial commit * issue page created using ActiveProject.tsx

* fix theme for footer * upadate color of footer

* refactor: implemented dynamic import on /people route * chore: removed useEffect in HoverCardWrapper * chore: removed useEffect from import in HoverCardWrapper * fix: small html fix

* experimental-commit-for-issue-forms * reverse chnages in footer

* enable/disable features from env variables * create featureIsEnabled function * add typesafety * update Readme --------- Co-authored-by: Rithvik Nishad <mail@rithviknishad.dev>

* init * fixes * added cron jobs * fixed routes * lint fix * comment envs * fix url verification challenge not working * some optimizations and support for showing manually added eod updates in app home * adds missing general updates in eod channel updates * Adds support for reacting to activity and clear updates message * correct cron schedule for utc tz * improve app home * update env * use consistent naming for envs --------- Co-authored-by: rithviknishad <mail@rithviknishad.dev>

…eviews to avoid rate limit (ohcnetwork#439) * improve message format * log only errors * optimize how reviews are pulled to avoid github's rate limit

* Added contrbutors OG image using genratemetadata * convert css to tailwind css and change logo.webp to .jpg and creat symlink * fix font url * og image modified to opengraph-image.tsx * merge conflicts and move opengraph-image.tsx to data repo * revert the changes og-image.tsx keep on leaderboard repo * revert the changes og-image.tsx keep on leaderboard repo * logo.png delted --------- Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com>

…#588/fix-logo Add logo.png to repository

…o GraduteAttribute page) (ohcnetwork#585)

…cnetwork#600)

netlify · 2025-01-09T08:11:21Z

👷 Deploy request for leaderboard-develop pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	`89594a7`

rajku-dev · 2025-01-10T02:18:07Z

How can i test the scraper for debugging? i can't run it on local machine?
@rithviknishad

rajku-dev · 2025-01-10T03:04:57Z

There were two directions for each fetched Fork

Fetch all its events and filter the PushEvent (REST event endpoint)
Fetch all its branches and get their commit history (GraphQL)

i went with the second one for now.

rithviknishad · 2025-01-10T04:13:52Z

Not a fan of N+1 approach. If this was deployed for a much bigger github org, which has too many forks, fetching commit history of each fork's branches might go off the limits pretty quickly.

Let's do a 2 part PR. Let's first get the org's repo commits tracked first. We can figure out something for the fork's. We only ideally care about commits pushed directly to default branch and commits pushed to PR branches alone.

This reverts commit ae0677a.

rajku-dev · 2025-01-10T05:10:04Z

Not a fan of N+1 approach. If this was deployed for a much bigger github org, which has too many forks, fetching commit history of each fork's branches might go off the limits pretty quickly.

Let's do a 2 part PR. Let's first get the org's repo commits tracked first. We can figure out something for the fork's. We only ideally care about commits pushed directly to default branch and commits pushed to PR branches alone.

I do agree with you, Please check this out #602 (comment)

scraper/src/github-scraper/parseEvents.ts

rajku-dev · 2025-01-15T03:33:39Z

Review is required in this PR, @rithviknishad
also the 2nd Part PR.
Please let me know if any changes are required.

kumar11jr and others added 30 commits April 10, 2024 19:57

Fix: Scroll in activity (ohcnetwork#425)

5046e6e

* Configurable favicon.ico and logo.webp * modified link * fix: side navbar collapse on link click * remove my comments made my me * "fix:Scroll in Activity of user "

hidden leaderboard filters in mobile view (ohcnetwork#387)

45faacd

* hidden leaderboard filters in mobile view * Added filter icon button

Adds support for listing good first issues (ohcnetwork#431)

5e7e4c1

* initial commit * issue page created using ActiveProject.tsx

fix theme for footer (ohcnetwork#420)

cf581ec

* fix theme for footer * upadate color of footer

refactor: implemented dynamic import on /people route (ohcnetwork#423)

b86034b

* refactor: implemented dynamic import on /people route * chore: removed useEffect in HoverCardWrapper * chore: removed useEffect from import in HoverCardWrapper * fix: small html fix

Adds issue forms (ohcnetwork#434)

31df48e

* experimental-commit-for-issue-forms * reverse chnages in footer

enable/disable features from env variables (ohcnetwork#418)

7824651

* enable/disable features from env variables * create featureIsEnabled function * add typesafety * update Readme --------- Co-authored-by: Rithvik Nishad <mail@rithviknishad.dev>

Fixes alignment of search icon in leaderboard (ohcnetwork#437)

b306449

fix scraper dry run

5c6144c

slack eod bot: fix incorrect env being used in update app home

1e9556d

slack eod bot: add logs

a70e1d1

slack eod bot: Increase max duration and await for sendMessage

bafe821

daily reports api: adds missing await

313af36

Slack EOD Bot: Prevent N+1 GraphQL calls to GitHub APIs for pulling r…

0882029

…eviews to avoid rate limit (ohcnetwork#439) * improve message format * log only errors * optimize how reviews are pulled to avoid github's rate limit

Remove cron from vercel config

4582ebb

slack eod bot: fix incorrect url

fe7c741

Use GH Actions to post EOD activities to slack (ohcnetwork#441)

2bc4806

update EOD updates API

3933218

eod activities: prevent collision of reserved names

0d803e2

eod activities workflow: fix incorrect script

07c9b48

fix script path

44ac3f6

fix imports

4b7751a

fix octokit import

4acfe87

add env for contributors dir

c10845b

slack eod activities: fix contributors dir path resolve

069d14b

fix incorrect dir

b4ed47b

update scripts to make it ready for action

1d9fa72

fix leaderboard profile url

16e212a

DonXavierdev and others added 11 commits December 31, 2024 14:19

Merge branch 'main' into issues/ohcnetwork#588/fix-logo

bf70bba

Merge pull request ohcnetwork#592 from DonXavierdev/issues/ohcnetwork…

461bbd6

…#588/fix-logo Add logo.png to repository

add relative time for push event on feed page (ohcnetwork#594)

24118eb

Incorrect Year according to Week Display on Leaderboard (ohcnetwork#589)

07783d1

PR Merge Icon corrected (ohcnetwork#584)

0c63e62

Replace relative imports with absolute imports (ohcnetwork#580)

f2850ae

FIXED: Leaderboard list responsivness (ohcnetwork#562)

ff28a60

Added Functionality to learn more in contributors page (Redirection t…

e87ea88

…o GraduteAttribute page) (ohcnetwork#585)

Add: Identify First-Time PR Contributors for Last Week (ohcnetwork#576)

46eda0a

Fix Auto Width Shrink Issues in discussions and releases page (oh…

ddbdd01

…cnetwork#600)

support for PushEvent for core-team members

115bd25

rajku-dev added 2 commits January 9, 2025 13:55

grap Commit Type from GitHub API docs

2f1e54f

enable tracking pushedCommits from forked repos

ae0677a

Revert "enable tracking pushedCommits from forked repos"

d7eaee9

This reverts commit ae0677a.

rajku-dev changed the title ~~Add support for tracking commits pushed to branch~~ Add support for tracking commits pushed to branch (core) Jan 10, 2025

rajku-dev mentioned this pull request Jan 10, 2025

Add support for tracking commits pushed to branch (Forked) #602

Open

rithviknishad reviewed Jan 10, 2025

View reviewed changes

scraper/src/github-scraper/parseEvents.ts Outdated Show resolved Hide resolved

rajku-dev added 2 commits January 10, 2025 13:37

exclude merge/cherry-pick commits

ac161fe

refine merge commit logic | change commit api link to github link

3cf8ad9

rajku-dev requested a review from rithviknishad January 11, 2025 08:13

rajku-dev added 3 commits January 11, 2025 13:43

Merge branch 'main' into issue/373/support-forked-commits

1db1d86

add stats tooltip | fix typo (ohcnetwork#607)

c525e38

Merge branch 'main' into issue/373/support-forked-commits

89594a7

rithviknishad changed the base branch from main to activities February 19, 2025 11:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add support for tracking commits pushed to branch (core) #601

Add support for tracking commits pushed to branch (core) #601

rajku-dev commented Jan 9, 2025 •

edited

Loading

netlify bot commented Jan 9, 2025 •

edited

Loading

rajku-dev commented Jan 10, 2025 •

edited

Loading

rajku-dev commented Jan 10, 2025 •

edited

Loading

rithviknishad commented Jan 10, 2025

rajku-dev commented Jan 10, 2025 •

edited

Loading

rajku-dev commented Jan 15, 2025 •

edited

Loading

Add support for tracking commits pushed to branch (core) #601

Are you sure you want to change the base?

Add support for tracking commits pushed to branch (core) #601

Conversation

rajku-dev commented Jan 9, 2025 • edited Loading

netlify bot commented Jan 9, 2025 • edited Loading

👷 Deploy request for leaderboard-develop pending review.

rajku-dev commented Jan 10, 2025 • edited Loading

rajku-dev commented Jan 10, 2025 • edited Loading

rithviknishad commented Jan 10, 2025

rajku-dev commented Jan 10, 2025 • edited Loading

rajku-dev commented Jan 15, 2025 • edited Loading

rajku-dev commented Jan 9, 2025 •

edited

Loading

netlify bot commented Jan 9, 2025 •

edited

Loading

rajku-dev commented Jan 10, 2025 •

edited

Loading

rajku-dev commented Jan 10, 2025 •

edited

Loading

rajku-dev commented Jan 10, 2025 •

edited

Loading

rajku-dev commented Jan 15, 2025 •

edited

Loading