Merge lp:~nigelbabu/launchpad/destroy-statusexplanation-88545 into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Abel Deuring | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 13908 | ||||
Proposed branch: | lp:~nigelbabu/launchpad/destroy-statusexplanation-88545 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
251 lines (+8/-62) 10 files modified
lib/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) |
||||
To merge this branch: | bzr merge lp:~nigelbabu/launchpad/destroy-statusexplanation-88545 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Abel Deuring (community) | code | Approve | |
Review via email: mp+74704@code.launchpad.net |
Commit message
[r=adeuring][bug=88545][incr] Nuke statusexplanation
Description of the change
= Description =
The term "status explanation" is now hidden, except in the activity log, where it causes confusion. Since it doesn't and will never serve any useful purpose, the database field should be destroyed. I've removed almost all references to it in tests as well as code.
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
./lib/lp/
1: narrative uses a moin header.
24: narrative uses a moin header.
54: narrative uses a moin header.
232: narrative uses a moin header.
248: want exceeds 78 characters.
258: narrative uses a moin header.
367: narrative uses a moin header.
433: narrative uses a moin header.
447: narrative uses a moin header.
491: narrative uses a moin header.
538: narrative uses a moin header.
make: *** [lint] Error 11
thanks for picking this up, Nigel, r=me.