Review Request 36: improve update syncs
Tim Flink
fedoraqa.devel at gmail.com
Wed Jul 17 16:59:13 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-tflink.rhcloud.com/r/36/#review42
-----------------------------------------------------------
blockerbugs/controllers/main.py
<http://reviewboard-tflink.rhcloud.com/r/36/#comment69>
this query has become rather complicated, could you add a quick comment about what it's doing?
blockerbugs/controllers/main.py
<http://reviewboard-tflink.rhcloud.com/r/36/#comment70>
A short comment on this query would also be nice
blockerbugs/templates/update_list.html
<http://reviewboard-tflink.rhcloud.com/r/36/#comment73>
I'd rather see these two tables be collapsed into a loop instead of having the code duplication
blockerbugs/templates/update_list.html
<http://reviewboard-tflink.rhcloud.com/r/36/#comment71>
This needs another empty <td></td> - you caught the other one, though
blockerbugs/templates/update_list.html
<http://reviewboard-tflink.rhcloud.com/r/36/#comment72>
you need quotes around the {{ bug.summary }} or else only the first word of the title will show up in the tooltip
A couple of minor nits but overall it looks good.
- Tim Flink
On July 15, 2013, 5:18 p.m., Ilgiz Islamgulov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard-tflink.rhcloud.com/r/36/
> -----------------------------------------------------------
>
> (Updated July 15, 2013, 5:18 p.m.)
>
>
> Review request for blockerbugs.
>
>
> Repository: blockerbugs
>
>
> Description
> -------
>
> Add partial updates sync.
> Display proposed/accepted fe/blocker bugs list associated with update on update list web page.
>
>
> Diffs
> -----
>
> testing/testfunc_update_sync.py 69f6bf8a459c71a9325cd633e7e33ce29354d7cd
> testing/test_updatesync_extract_information.py 1fc701ef79a74e27cf8223b56a04d63025b6509f
> testing/test_controllers.py f351832ad4350fcff3d67c0137ff62892639e908
> blockerbugs/util/update_sync.py d664839ec1c5979dce980e7baad58154f4622e11
> blockerbugs/util/bug_sync.py 49cce49740cd6f5b1f430f58c8d1b522e1f0b7e3
> blockerbugs/templates/update_list.html a83dd9a422e51ea58e04ae58acd8fefe6eb1f79f
> blockerbugs/models/update.py 2ee3ca7ee62bd55bbd71555c13fecfab504c4f07
> blockerbugs/controllers/main.py 4f23e35558ac3f8ed2ccc5d897b2219e449e7277
> blockerbugs/cli.py 694736d098b6285c53db91f91350360d478c4229
>
> Diff: http://reviewboard-tflink.rhcloud.com/r/36/diff/
>
>
> Testing
> -------
>
> Wrote test suites.
> I've tested on my develop instance.
>
>
> Thanks,
>
> Ilgiz Islamgulov
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.fedoraproject.org/pipermail/qa-devel/attachments/20130717/68e6d4be/attachment-0001.html>
More information about the qa-devel
mailing list