Skip to content

feat: contributor graphs#45

Merged
Anshgrover23 merged 15 commits intotscircuit:mainfrom
ArnavK-09:main
Feb 4, 2025
Merged

feat: contributor graphs#45
Anshgrover23 merged 15 commits intotscircuit:mainfrom
ArnavK-09:main

Conversation

@ArnavK-09
Copy link
Copy Markdown
Collaborator

@ArnavK-09 ArnavK-09 commented Feb 3, 2025

/claim #44

image

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
contribution-tracker ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 3, 2025 8:14pm

Copy link
Copy Markdown
Contributor

@Anshgrover23 Anshgrover23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ArnavK-09 why the graph is in decreasing order of dates,{<-----------------} like this, the dates should be like this -----------------> in increasing order and also
Please keep the axis at a max of 25, i don't want the graph to constantly scale to a user's maximum as we change users. this is also mentioned in issue.
image

@Anshgrover23
Copy link
Copy Markdown
Contributor

@ArnavK-09 ohh, why u took the y-axis to bountied amount???

@ArnavK-09
Copy link
Copy Markdown
Collaborator Author

@ArnavK-09 ohh, why u took the y-axis to bountied amount???

Y axis is dynamic, it's based on every stats

@Anshgrover23
Copy link
Copy Markdown
Contributor

@ArnavK-09 attach the latest screenshot in description.

@seveibar
Copy link
Copy Markdown
Contributor

seveibar commented Feb 3, 2025

@ArnavK-09 yea the Y axis is basically not readable when you combine all these things. If you want to show everything, you should have a dropdown that allows different fields to be filtered

e.g. "Reviews", "Merges", "Issues", "Bounties", "Contributor Score"

You can keep the dynamic axis if you add this dropdown, make sure it defaults to "Merges" since that's the primary thing i look at for sponsorships (i also look at contribution score, so maybe that should be first but merges is important)

Reviewed at 16m https://www.youtube.com/watch?v=pQ8Vj7HPJwA

@ArnavK-09
Copy link
Copy Markdown
Collaborator Author

@ArnavK-09 attach the latest screenshot in description.

Done

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good but the bottom dates should be in the format "MONTH/DAY"

Also i would remove the "skeleton data" checkbox, it's not helpful enough to justify the space on the screen imo

@Anshgrover23 Anshgrover23 merged commit 4dc0f4d into tscircuit:main Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants