+localpackagediffs page should show who made the change rather than sponsor

Bug #798865 reported by Bryce Harrington on 2011-06-17
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Gavin Panella

Bug Description

On https://launchpad.net/ubuntu/oneiric/+localpackagediffs the Uploader column should represent the person listed as the 'Changed-By' rather than the sponsor. (Technically, the latter was the actual uploader of the change, but the former is the one that gets credit for the upload and is the contact for any issues that arise due to the change.)

From: Julian Edwards <email address hidden>
To: <email address hidden>

On Thursday 16 June 2011 18:11:01 Micah Gersten wrote:
> On 06/16/2011 04:26 AM, Dave Walker wrote:
> > <snip />
> > * The new page should probably indicate the sponsor of the Ubuntu
> > upload (if present).
> > <snip />
> > Thanks again!
> >
> > Kind Regards,
> > Dave Walker
> Actually, the "uploader" on this page is the sponsor. It should really
> show the person who got credit for the upload as that person is
> considered TIL.

Can you file a bug about that please and we'll fix it. Presumably it should
be the Changed-By instead?

Related branches

Bryce Harrington (bryce) on 2011-06-17
tags: added: derivation
Dave Walker (davewalker) wrote :

I think there needs to be /some/ indication that it was a sponsored upload. A sponsored upload, to me, could indicate that the "touched it last" convention doesn't apply as tightly. Additionally, the sponsor takes most of the responsibility if something bad happens. :)

If it doesn't burn too much screen space, i'd suggest showing both fields if suitable.


Bryce Harrington (bryce) wrote :

Good point, I agree that's useful info to indicate in some fashion or other.

Maybe an icon could be used, with hoverover to see the sponsor's name?

https://merges.ubuntu.com/main.html shows the sponsor in a smaller font underneath the changer's name. Dunno if that is stylistically allowed in Ubuntu, but if it is perhaps that could be another option.

Given how many columns are already present in the table, adding another column for sponsor would not be desirable, I'd guess.

Changed in launchpad:
status: New → Triaged
importance: Undecided → High
Gavin Panella (allenap) on 2011-07-22
Changed in launchpad:
status: Triaged → In Progress
assignee: nobody → Gavin Panella (allenap)
summary: - +localpackagediffs page should show uploader rather than sponsor
+ +localpackagediffs page should show who made the change rather than
+ sponsor
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Gavin Panella (allenap) on 2011-07-26
tags: added: qa-ok
removed: qa-needstesting
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
removed: qa-ok
William Grant (wgrant) on 2011-07-26
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant) on 2011-07-27
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers