Opened 3 years ago

Last modified 3 years ago

#5592 new enhancement

PR's/patches on Trac or Github doesn't make your profile "public"

Reported by: rolfsiebers's profile rolfsiebers Owned by:
Milestone: Priority: normal
Component: Profiles Keywords:


A lot of people who only made a PR on Github, commented on a Trac ticket or added Patch on Trac bio's on their profile aren't visible yet because they haven't met the requirements yet for a public bio.

Is this something we also would like to add for the requirements? Any thoughts on this?

Change History (6)

#1 @rolfsiebers
3 years ago

Maybe we could show that as an activity on their profile?

#2 @SergeyBiryukov
3 years ago

  • Component changed from Site to Profiles

#4 @dd32
3 years ago

Maybe we could show that as an activity on their profile?

I feel like this is the way to go.

Using the existing webhook, we should be able to add entries for "Submitted a PR", "Commented on a PR", "Submitted an issue" (for other repo's), etc.

Ideally, I think we'd extend that to every repo in the WordPress GitHub organisation rather than just the core PRs.

#5 @dd32
3 years ago

In 11270:

Profiles: Display GitHub activity on profiles.

This is implemented through a WordPress GitHub organization webhook, which parses pushed events and stores it in a table for later querying.
This is how Trac/SVN (and now GitHub) activity is inserted into the profiles activity stream.

Issues created & closed, Pull Requests submitted & merged, and pushes to repositories are supported.

Activity is only recorded against a profile if their GitHub profile is linked to, but will show up once that's done.

This is still a work in progress, and we may want to import older history as well.

See #4447, #5592.

#6 @dd32
3 years ago

Status update on this ticket: PRs will now trigger visibility, as will profile badges (#5593). Patches uploaded to Trac are not registered on Profiles.

I'm tempted to suggest that files uploaded to Trac shouldn't need to be tracked, but that commit props should be recorded on profiles instead.

Note: See TracTickets for help on using tickets.