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
Merged
into
lp:launchpad
- 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 |
Changed in rosetta: | |
status: | New → Triaged |
importance: | Undecided → Low |
Changed in rosetta: | |
assignee: | nobody → Adi Roiban (adiroiban) |
Changed in rosetta: | |
status: | Triaged → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
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.