-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
WIP: overhaul profile overview tab UI #27183
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
base: main
Are you sure you want to change the base?
Conversation
I've marked this as a WIP because as multiple others mentioned, pinning repositories would be better. The feature works as-is, however. If it's preferred for pinning repositories to be added as a separate follow-up PR, I'd be open to that as well :) |
Pinning would be fantastic! The best place to look is the previous pin-issue PR ( #19831 ) as there is alot of discussion as well as some code ideas. Likewise the merged "pin issues" PR ( #24406 ) should proivde alot of reuse as well as what can be done with the DB |
I has try it in #12498, may be helpfull for you |
and #19831 |
@hazycora i did resolve the conflicts ... and if you make it a "profile overview" enhancement only e.g. remove the grid and do that in another pull. then we can get that enhancemen in sooner :) |
There was some love shown for this idea in the Discord so I thought I'd make a draft PR for it. This is a patch to make the overview tab of user profiles display not just a profile readme but also a users 6 most popular repos (sorted by stars), along with their activity. This PR also does the same for organization overview pages (albeit without activity) building off of my other PR.
People suggested on Discord that pinning repositories would be a better solution than displaying the 6 most starred repositories. I completely agree, and I'll look into adding that, but as the pinned repositories list would need to be stored in the database some guidance on how best to do that would be appreciated!