Don't display merged accounts
Bug #512307 reported by
Adi Roiban
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Adi Roiban | ||
Bug Description
On https:/
We should not show merged accounts.
In bug 121520 , we fixed some placed where merged accounts were displayed, but missed language-
------
Also maybe we can refactor the TAL code into a macro.
Related branches
lp:~adiroiban/launchpad/bug-512307
- Graham Binns (community): Approve (code)
-
Diff: 246 lines (+111/-19)5 files modifiedlib/lp/translations/browser/language.py (+26/-3)
lib/lp/translations/browser/tests/language-views.txt (+73/-7)
lib/lp/translations/stories/standalone/xx-pofile-details.txt (+4/-3)
lib/lp/translations/templates/language-portlet-top-contributors.pt (+2/-2)
lib/lp/translations/templates/pofile-details.pt (+6/-4)
| Changed in rosetta: | |
| assignee: | nobody → Adi Roiban (adiroiban) |
| Changed in rosetta: | |
| assignee: | Adi Roiban (adiroiban) → nobody |
Revision history for this message
| Adi Roiban (adiroiban) wrote | #1 |
| Changed in rosetta: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| Changed in rosetta: | |
| assignee: | nobody → Adi Roiban (adiroiban) |
| Changed in rosetta: | |
| status: | Triaged → In Progress |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #2 |
| Changed in rosetta: | |
| milestone: | none → 10.03 |
| status: | In Progress → Fix Committed |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Curtis Hovey (sinzui) wrote Bug 512307 Fix released | #3 |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Jeroen suggest that filtering will leave out contributors.
Instead of merged account we should display the target account.
I have submitted a fix but got stuck in writing a working test case.