Bug 1286979 - Add a way to trigger a hook from web interface (#126)

* Bug 1286979 - Add a button to trigger a hook

* Bump taskcluster-client to 1.1.0

* Fix style issues in hookeditor

hooks/hookeditor.jsx

@@ -185,6 +185,9 @@ var HookDisplay = React.createClass({
  onClick={this.props.startEditing}>
  <bs.Glyphicon glyph="pencil"/>&nbsp;Edit Hook
  </bs.Button>
+ <bs.Button bsStyle="success" onClick={this.props.triggerHook}>
+ <bs.Glyphicon glyph="repeat" /> Trigger Hook
+ </bs.Button>
  </bs.ButtonToolbar>
  </div>
  }
@@ -533,6 +536,7 @@ var HookEditView = React.createClass({
  currentHookGroupId: React.PropTypes.string,
  refreshHookList: React.PropTypes.func.isRequired,
  selectHook: React.PropTypes.func.isRequired,
+ triggerHook: React.PropTypes.func.isRequired,
  },
 
  getInitialState() {
@@ -602,14 +606,22 @@ var HookEditView = React.createClass({
  return <HookDisplay hook={this.state.hook}
  currentHookId={this.props.currentHookId}
  currentHookGroupId={this.props.currentHookGroupId}
- startEditing={this.startEditing} />
+ startEditing={this.startEditing}
+ triggerHook={this.triggerHook} />
  }
  },
 
  startEditing() {
  this.setState({editing: true});
  },
 
+ triggerHook() {
+ // Payloads are ignored, so we send empty data over
+ this.hooks
+ .triggerHook(this.props.currentHookGroupId, this.props.currentHookId, {})
+ .catch((error) => this.setState({ error }));
+ },
+
  createHook(hookGroupId, hookId, hook) {
  // add hookId and hookGroupId to the hook, since they are required
  // by the schema
@@ -633,7 +645,7 @@ var HookEditView = React.createClass({
  this.props.currentHookGroupId,
  this.props.currentHookId,
  hook
- )
+ );
  this.setState({
  hook: stripHookIds(hook),
  editing: false,

package.json

@@ -68,7 +68,7 @@
  "rison": "^0.1.1",
  "shell-escape": "^0.2.0",
  "slugid": "^1.1.0",
- "taskcluster-client": "^0.23.17",
+ "taskcluster-client": "^1.1.0",
  "term.js": "0.0.4",
  "webworkify": "^1.2.1",
  "www-authenticate": "^0.6.2",