(in API) Person.logo_link is hard to use and performs poorly
Bug #713873 reported by
Robert Collins
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Triaged
|
High
|
Unassigned | ||
| LoCo Team Portal |
Triaged
|
Low
|
Unassigned | ||
Bug Description
mugshot_link too - see bug 340640. At the moment we output a reference to the mugshot object of the Person which entails another lookup to Launchpad to get that page, which issues a 302 to the librarian.
We should just output librarian urls directly.
Related branches
lp://staging/~nigelbabu/launchpad/logo-links-713873
- Curtis Hovey (community): Disapprove
- Raphaël Badin (community): Needs Fixing
-
Diff: 45 lines (+24/-0)2 files modifiedlib/lp/registry/interfaces/person.py (+16/-0)
lib/lp/registry/model/person.py (+8/-0)
| summary: |
- Person.logo_link is hard to use and performs poorly + (in API) Person.logo_link is hard to use and performs poorly |
Revision history for this message
| Adnane Belmadiaf (daker) wrote | #1 |
| Changed in loco-directory: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| tags: | added: easy |
To post a comment you must log in.
I am adding to LTP so we can keep track the work being done on this.