On Dec. 10, 2013, 5:58 p.m., Tim Flink wrote:
> blockerbugs/__init__.py, line 162
>
<
http://reviewboard-tflink.rhcloud.com/r/57/diff/1/?file=1605#file1605line...
>
> Bah, have we really been running this whole time with unclosed <span> tags
on the update status labels?
Yes :)
On Dec. 10, 2013, 5:58 p.m., Tim Flink wrote:
> blockerbugs/models/update.py, line 55
> <
http://reviewboard-tflink.rhcloud.com/r/57/diff/1/?file=1607#file1607line...
>
> Why is this sort order change an improvement? If an older update is pending
while a newer update is stable, we'd want to show the newer update
But wouldn't the older update be marked as obsolete?
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-tflink.rhcloud.com/r/57/#review118
-----------------------------------------------------------
On Dec. 10, 2013, 3:54 p.m., Martin Krizek wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-tflink.rhcloud.com/r/57/
-----------------------------------------------------------
(Updated Dec. 10, 2013, 3:54 p.m.)
Review request for blockerbugs.
Bugs: 438
https://fedorahosted.org/fedora-qa/ticket/438
Repository: blockerbugs
Description
-------
commit 911781af9cc06e27c0a8580f07f7673c90e7329d
Author: Martin Krizek <mkrizek(a)redhat.com>
Date: Tue Dec 10 16:28:52 2013 +0100
Sort updates by (status.desc(), pending.desc())
This way an update label is correctly set when multiple updates fix a
bug. The label is chosen by the following order (the lowest one):
pending testing < testing < pending stable < stable
Fixes: #438
Note: the rest of the issues with displaying label is handled outside this ticket in
https://phab.qadevel.cloud.fedoraproject.org/T21
Diffs
-----
blockerbugs/models/update.py 89c083da5c91558d4dead118c67e4a1138b8cae1
blockerbugs/controllers/main.py 1795ed1d717bb85e0f3d9e7e7008d68c2aae36d8
blockerbugs/__init__.py dcf6c5aadb24e6d86e95d5bb3d608b81721de47f
Diff:
http://reviewboard-tflink.rhcloud.com/r/57/diff/
Testing
-------
Sorting seems to be working as expected.
Thanks,
Martin Krizek