Merge branch 'build-notification-null' into 'master'

Adds a null check to build notifications Closes #27948 See merge request !9108

Merge branch 'build-notification-null' into 'master'
d3496c45 · Filipa Lacerda · Regis · 42cce7d1 · d3496c45
Commit d3496c45 authored Feb 09, 2017 by Filipa Lacerda Committed by Regis Feb 14, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

merge_request_widget.js.es6 app/assets/javascripts/merge_request_widget.js.es6 +1 -1

No files found.
--- a/app/assets/javascripts/merge_request_widget.js.es6
+++ b/app/assets/javascripts/merge_request_widget.js.es6
@@ -154,7 +154,7 @@ require('./smart_interval');
      return $.getJSON(this.opts.ci_status_url, (function(_this) {
        return function(data) {
          var message, status, title;
-          if (data.status === '') {
+          if (!data.status) {
            return;
          }
          if (data.environments && data.environments.length) _this.renderEnvironments(data.environments);