Merge pull request #90 from marmeladema/checkbox-disabled

Disable rollup checkbox if PR is not approved

Author: pietroalbini

homu/html/queue.html

@@ -166,7 +166,7 @@ <h1>Homu queue - {% if repo_url %}<a href="{{repo_url}}" target="_blank">{{repo_
                 {% for state in states %}
                 <tr class="{{state.greyed}}">
                     <td class="hide">{{loop.index}}</td>
-                    <td><input type="checkbox" data-num="{{state.num}}"></td>
+                    <td><input type="checkbox" data-num="{{state.num}}" {{ '' if state.status == 'approved' else 'disabled' }}></td>
                     {% if multiple %}
                     <td><a href="{{state.repo_url}}">{{state.repo_label}}</a></td>
                     {% endif %}

homu/main.py

@@ -719,7 +719,8 @@ def parse_commands(body, username, user_id, repo_label, repo_cfg, state,
                         data={
                             'selected': repo_cfg['buildbot']['builders'],
                             'comments': INTERRUPTED_BY_HOMU_FMT.format(int(time.time())),  # noqa
-                    })
+                        }
+                    )
 
             if 'authzfail' in res.text:
                 err = 'Authorization failed'