As of March 2015 or so, we are now capturing both individuals and companies who've contributed to the closing of an issue in the Drupal.org database, in addition to in commit messages. However, we don't currently expose this data publicly. One reason for this is that we don't have historical data, especially of organization/customer contributions.

I talked to Dries about this problem and he feels strongly we should move forward with exposing this data despite the lack of historical data, and that it is different from commit credit and thus can be visually treated differently.

If we set the timeframe for something like ~3 months:

a) we don't run into problems with lack of historical data
b) it's a more useful metric anyway because it indicates current activity (which should be ranked more prominantly than aggregate historical activity anyway, especially in e.g. marketplace listings)
c) it encourages organizations to continuously contribute back on an active and ongoing basis, versus perpetually getting credit for things that happened far into the past.

Preview

https://ci-drupal.redesign.devdrupal.org/u/tim.plunkett
https://ci-drupal.redesign.devdrupal.org/u/lks90
Screenshot
https://ci-drupal.redesign.devdrupal.org/marketplace/acquia
https://ci-drupal.redesign.devdrupal.org/marketplace/md-systems
https://ci-drupal.redesign.devdrupal.org/drupal-association

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drumm’s picture

Title: Expose issue credits data on user / organization profiles » Expose recent issue credits data on user & organization profiles
webchick’s picture

Issue summary: View changes

Adding an HR to separate what Dries said vs. what I came up with ad-hoc while writing the issue summary. :P

webchick’s picture

Issue summary: View changes

Some issue summary updates.

drumm’s picture

Assigned: Unassigned » drumm
drumm’s picture

Issue summary: View changes

I edited the language in the mockup to be a bit more concise and less formal.

  • drumm committed 4e0cf32 on 2498497-recent-credit
    Issue #2498497: Expose recent issue credits data on user...
drumm’s picture

drumm’s picture

(dev sites will under-count because older issues are truncated.)

tvn’s picture

I wonder if "Contributed to fixing X..." is a better wording. 'Closing' might give an impression we are talking not only about fixed issues, but also things like 'closed(won't fix), issue triage stuff.

  • drumm committed 06da36c on 2498497-recent-credit
    Issue #2498497: Expose recent issue credits data on organization pages
    
  • drumm committed a0a7d18 on 2498497-recent-credit
    Issue #2498497 by tvn: Expose recent issue credits data on user profiles...
drumm’s picture

Issue summary: View changes

Replacing closing with Fixing sounds like a good improvement to me, done.

And this is set up for organization pages now, examples at https://ci-drupal.redesign.devdrupal.org/marketplace/acquia, https://ci-drupal.redesign.devdrupal.org/marketplace/md-systems, and https://ci-drupal.redesign.devdrupal.org/drupal-association

drumm’s picture

Status: Active » Needs review

joshuami reviewed this and asked for the links to go to listings of the specific issues credited. The specific information will help with QA and tracking down any abuse, too. And we could use more-broad UX work on user profiles, that's a separate issue.

The basics are there, so setting this to needs review, I'll be adding those other pages.

  • drumm committed 90f8122 on 2498497-recent-credit
    Issue #2498497 by drumm, webchick, tvn: Expose recent issue credits data...
drumm’s picture

YesCT’s picture

wait, isn't that a list of just the "recent" ones?
I think it should be if those are the links from the "recent" block.

If the link comes from
"Contributed to fixing 70 issues in the past 3 months
Drupal core 48 issues"

then the link should like to just those 48 issues.

That should probably have a title which says something about the time frame those are from.

And maybe then also a link to All for ever (which wont be all forever until we do #2042697: Add historical issue credits to Drupal.org user profile

davidwbarratt’s picture

I think the test "Page Manager" should link to https://www.drupal.org/project/page_manager and the text "1 issue" should link to the issue list. The reason why is that I expected "Page Manager" to take me to the module page just like the "Projects" below. I think it should be consistent.

LoMo’s picture

+1 for Having project titles link to the project (and separately linking the 'x issues')

And while I do agree that starting with a 3-month window is appropriate, since we lack historical data, I think we should still show historical contributions, especially for organizations, at least as that data becomes available. (Maybe that's the idea, anyway?)

Perhaps in a year, the text could say "in the past year", instead of "in the past 3 months". Why? For example, some companies may blow their contrib budget on hosting a bigger Sprint once or twice a year or something; if we never show more than a 3-month window, it may not give as clear a picture of an organization's ongoing commitment to contribution.

That said, it's great to see something concrete coming from all the discussion of organizational attribution! :-)

joshuami’s picture

I see the points raised. I think we could launch with the list being "all credits recorded" and it wouldn't hurt much, but the confusing that clicking on the project title in two contexts creates could be a bit of a blocker.

What if the project title was still a link to the project, but the "x issues" count next to the title was formatted as a label that linked to the view that was a list of the issue credits? This could even match the appearance of the current "credit" labels on comments to reuse a pattern.

Issue credit counts as labels

Here is a very quick mockup of what the format could look like—realizing that the grey is too dark and it looks a bit different than the credit labels on comments :)

davidwbarratt’s picture

Yep I think that looks good, or rather, I understand what I'm clicking on by looking at it.

Also, I think that it should show all data we have a history for as well, but I assumed that was the intention.

drumm’s picture

Issue summary: View changes
FileSize
12.33 KB

I restored the links to projects and separate link to the credited issues. I went with a bit lighter styling, just a comma to separate the two links.

Screenshot

drumm’s picture

wait, isn't that a list of just the "recent" ones?
I think it should be if those are the links from the "recent" block.

#2042697: Add historical issue credits to Drupal.org user profile should be combined into the work here, so we don't have another list of projects. Maybe something like "Drupal core, 48 recent issues, 142 total" in each line; or something more visual if we can do it.

There can be only one list of issues credited, so we don't have 2 pages that are quite similar. Since the page is a bit buried, I think it is okay to show all of what we have, even if it isn't a complete history. We could put a note on the page if needed.

  • drumm committed 6c36a52 on 2498497-recent-credit
    Issue #2498497: Expose recent issue credits data on user...

  • drumm committed 06da36c on 7.x-3.x
    Issue #2498497: Expose recent issue credits data on organization pages
    
  • drumm committed 4e0cf32 on 7.x-3.x
    Issue #2498497: Expose recent issue credits data on user...
  • drumm committed 6c36a52 on 7.x-3.x
    Issue #2498497: Expose recent issue credits data on user...
  • drumm committed 90f8122 on 7.x-3.x
    Issue #2498497 by drumm, webchick, tvn: Expose recent issue credits data...
  • drumm committed a0a7d18 on 7.x-3.x
    Issue #2498497 by tvn: Expose recent issue credits data on user profiles...
drumm’s picture

Status: Needs review » Fixed
Issue tags: +needs drupal.org deployment

I believe this is now ready to deploy. Any non-urgent followups can go to #2042697: Add historical issue credits to Drupal.org user profile.

drumm’s picture

Issue tags: -needs drupal.org deployment

This is now deployed.

alexpott’s picture

alexpott’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

YesCT’s picture

YesCT’s picture

some contribution credit issues have d.o profile improvements tag, and some have nothing and are easy to get lost (and not about profiles), so tagging to organize credit ones.

YesCT’s picture

#2491873: Add commits to user.json API endpoint is related to the historical issue, but I suspect we have a json problem in general with exposing credits. Shall we use that issue, or is there another?

drumm’s picture

The new credits are in the RestWS API, they are all fields, so they get exposed automatically. They are seriously lacking in documentation, the way they are stored is not obvious. Briefly, it is:

https://www.drupal.org/api-d7/node/2498497.json has field_issue_credit referencing comments by credited people.
https://www.drupal.org/api-d7/comment/9980901.json has field_attribute_as_volunteer. If that has a 0 value, then field_attribute_contribution_to and field_for_customer have relevant information.