translation sharing-details page should not require more than two roundtrips
Bug #817522 reported by
Aaron Bentley
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Aaron Bentley | ||
Bug Description
One roundtrip would be ideal, but with two you can perform a change and reload the view's JSON model.
Related branches
lp:~abentley/launchpad/translations-sharing
- j.c.sackett (community): Approve
-
Diff: 427 lines (+152/-118)3 files modifiedlib/lp/translations/javascript/sourcepackage_sharing_details.js (+49/-91)
lib/lp/translations/javascript/tests/test_sourcepackage_sharing_details.html (+1/-0)
lib/lp/translations/javascript/tests/test_sourcepackage_sharing_details.js (+102/-27)
lp:~abentley/launchpad/fix-lpviews
- Brad Crittenden (community): Approve (code)
-
Diff: 55 lines (+17/-5)2 files modifiedlib/lp/registry/browser/configure.zcml (+1/-0)
lib/lp/registry/browser/tests/test_productseries_views.py (+16/-5)
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-bad removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-bad |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| status: | Fix Released → Fix Committed |
| tags: |
added: bad-commit-13548 qa-bad removed: qa-ok |
Revision history for this message
| William Grant (wgrant) wrote | #3 |
Revision history for this message
| William Grant (wgrant) wrote | #4 |
| tags: |
added: qa-ok removed: qa-bad |
Revision history for this message
| William Grant (wgrant) wrote | #5 |
| tags: | removed: bad-commit-13548 |
| tags: | added: bad-commit-13542 |
| tags: |
added: bad-commit-13548 removed: bad-commit-13542 |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r13548 <http://