Abolish the "statusexplanation" database field
Bug #88545 reported by
Matthew Paul Thomas
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Nigel Babu | ||
Bug Description
When I originally proposed the "status explanation" field for bug reports, I thought it would be a way of presenting information that was relevant only to one particular project affected by the bug, without spamming people who weren't interested in that project. But this could never work, because there are no bugtask-specific lists of subscribers, so everyone gets mailed regardless (see bug 3529).
The term "status explanation" is now hidden, except in the activity log, where it causes confusion because it makes some bug comments show up and others not (see bug 88531). Since it doesn't and will never serve any useful purpose, the database field should be abolished.
See also bug 57414.
Related branches
lp://staging/~nigelbabu/launchpad/destroy-statusexplanation-88545
- Abel Deuring (community): Approve (code)
-
Diff: 251 lines (+8/-62)10 files modifiedlib/lp/bugs/browser/bugtask.py (+2/-10)
lib/lp/bugs/configure.zcml (+0/-2)
lib/lp/bugs/doc/bugtask-expiration.txt (+0/-7)
lib/lp/bugs/interfaces/bugtask.py (+0/-5)
lib/lp/bugs/model/bugtask.py (+1/-4)
lib/lp/bugs/scripts/bugexpire.py (+1/-2)
lib/lp/bugs/templates/bugtask-view.pt (+0/-10)
lib/lp/coop/answersbugs/emailtemplates/question-linked-bug-status-updated.txt (+0/-1)
lib/lp/coop/answersbugs/notification.py (+3/-15)
lib/lp/coop/answersbugs/tests/notifications-linked-bug.txt (+1/-6)
lp://staging/~nigelbabu/launchpad/kill-statusexplanation
- Stuart Bishop (community): Approve (db)
- Robert Collins: Pending requested
-
Diff: 174 lines (+68/-58)5 files modifieddatabase/sampledata/current-dev.sql (+28/-28)
database/sampledata/current.sql (+28/-28)
database/schema/comments.sql (+0/-1)
database/schema/fti.py (+0/-1)
database/schema/patch-2208-84-1.sql (+12/-0)
- Launchpad code reviewers: Pending requested
-
Diff: 1072 lines (+486/-414)7 files modifieddatabase/sampledata/current-dev.sql (+229/-209)
database/sampledata/current.sql (+224/-202)
database/schema/comments.sql (+1/-1)
database/schema/fti.py (+0/-1)
database/schema/patch-2208-82-1.sql (+19/-0)
database/schema/patch-2208-84-1.sql (+12/-0)
database/schema/preflight.py (+1/-1)
lp://staging/~nigelbabu/launchpad/the-return-of-destory-statusexplanation-88545
- William Grant (community): Approve (code)
-
Diff: 153 lines (+16/-25)2 files modifiedlib/canonical/launchpad/scripts/ftests/test_oops_prune.py (+12/-18)
lib/canonical/launchpad/scripts/oops.py (+4/-7)
lp://staging/~nigelbabu/launchpad/kill-statusexplanation-field-forever
- Stuart Bishop (community): Approve (db)
- Jeroen T. Vermeulen (community): Approve
- Robert Collins: Pending (db) requested
-
Diff: 174 lines (+68/-58)5 files modifieddatabase/sampledata/current-dev.sql (+28/-28)
database/sampledata/current.sql (+28/-28)
database/schema/comments.sql (+0/-1)
database/schema/fti.py (+0/-1)
database/schema/patch-2208-84-1.sql (+12/-0)
| description: | updated |
| description: | updated |
Revision history for this message
| Diogo Matsubara (matsubara) wrote | #1 |
| description: | updated |
| Changed in malone: | |
| status: | New → Triaged |
Revision history for this message
| Druciferre (drewchapin) wrote | #2 |
| Changed in malone: | |
| status: | Triaged → Invalid |
| Changed in malone: | |
| status: | Invalid → Triaged |
| importance: | Undecided → Low |
Revision history for this message
| Robert Collins (lifeless) wrote | #3 |
| summary: |
- Abolish the "status explanation" database field + Abolish the "statusexplanation" database field |
| Changed in launchpad: | |
| importance: | Low → High |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| Changed in launchpad: | |
| assignee: | nobody → Nigel Babu (nigelbabu) |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: | added: bad-commit-10978 |
Revision history for this message
| William Grant (wgrant) wrote | #6 |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| William Grant (wgrant) wrote | #7 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #8 |
| tags: |
added: qa-untestable removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #9 |
| tags: |
added: qa-needstesting removed: qa-untestable |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Nigel Babu (nigelbabu) wrote | #10 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Gavin fixed bug #249890 which removed the status explanation from the UI.