| Summary: | queues.webkit.org should display when a bot last rebooted | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||
| Component: | New Bugs | Assignee: | Eric Seidel (no email) <eric> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | abarth, commit-queue, mihaip, ojan | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Other | ||||||
| OS: | OS X 10.5 | ||||||
| Attachments: |
|
||||||
|
Description
Eric Seidel (no email)
2011-04-14 13:01:06 PDT
Created attachment 89632 [details]
Patch
Comment on attachment 89632 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=89632&action=review > Tools/QueueStatusServer/templates/queuestatus.html:12 > +Last Pass: {{ last_pass.date|timesince }} ago You may be able to move this to a sub-template that you then use {% with "foo" as param %} {%include "summary_snippet.html" %} {% endwidth %} However, this is kind of awkward pre-Django 1.3 (where you can specify the parameters directly in the include tag: http://docs.djangoproject.com/en/dev/ref/templates/builtins/?from=olddocs#include), since you need a with statement per parameter. I'd say the repetition is fine for just two places. Comment on attachment 89632 [details] Patch Clearing flags on attachment: 89632 Committed r83956: <http://trac.webkit.org/changeset/83956> All reviewed patches have been landed. Closing bug. |