Remove details link from productseries pages
Bug #737418 reported by
Henning Eggers
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Henning Eggers | ||
Bug Description
Productseries translation pages (and template pages on productseries) have a link to the sharing details page in the "Sharing Information" portlet. This was placed there because originally it was planned to have such a page for productseries as we have now for a source package. But since enabling upstream translation sharing is mostly in the interest of Ubuntu translators, such a page for productseries would not make much sense especially since the work flow for an existing upstream project is slightly different.
Anyway, it follows that the links to "setup message sharing" need to be removed and when upstream message sharing is configured it should point to the details page for the sourcepackage.
Related branches
lp://qastaging/~henninge/launchpad/devel-737418-remove-sharing-links
- Graham Binns (community): Approve (code)
-
Diff: 363 lines (+86/-82)6 files modifiedlib/lp/translations/browser/potemplate.py (+3/-9)
lib/lp/translations/browser/productseries.py (+3/-5)
lib/lp/translations/browser/sourcepackage.py (+6/-12)
lib/lp/translations/browser/tests/test_sharing_information.py (+56/-40)
lib/lp/translations/browser/translationsharing.py (+16/-15)
lib/lp/translations/stories/standalone/xx-potemplate-index.txt (+2/-1)
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Henning Eggers (henninge) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.04 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Deryck Hodge (deryck) wrote | #2 |
| tags: | added: upstream-translations-sharing |
Revision history for this message
| Robert Collins (lifeless) wrote | #3 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Henning Eggers (henninge) wrote | #4 |
Revision history for this message
| Deryck Hodge (deryck) wrote | #5 |
To post a comment you must log in.
Fixed in stable r12690 <http://