Bug:+subscribe should have its own view
Bug #651240 reported by
Graham Binns
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Graham Binns | ||
Bug Description
Bug:+subscribe is actually handled by BugTaskView, for no apparent reason. The +subscribe functionality should be extracted from BugTaskView so that we can implement the better-
Related branches
lp:~gmb/launchpad/extract-subscription-view-bug-651240
- Abel Deuring (community): Approve (code)
-
Diff: 618 lines (+276/-205)7 files modifiedlib/lp/bugs/browser/bugsubscription.py (+257/-2)
lib/lp/bugs/browser/bugtask.py (+7/-197)
lib/lp/bugs/browser/configure.zcml (+1/-1)
lib/lp/bugs/stories/bugs/xx-bug-personal-subscriptions.txt (+2/-2)
lib/lp/bugs/templates/bug-subscription.pt (+1/-1)
lib/lp/registry/browser/person.py (+7/-1)
lib/lp/registry/stories/person/xx-person-subscriptions.txt (+1/-1)
Revision history for this message
| Robert Collins (lifeless) wrote Re: [Bug 651240] [NEW] Bug:+subscribe should have its own view | #1 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #2 |
| Changed in malone: | |
| milestone: | none → 10.11 |
| tags: | added: qa-needstesting |
| Changed in malone: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in malone: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
It will also nicely remove some overhead from +index :)