Features/disable alert configs #3416

bthesorceror · 2018-05-10T16:37:16Z

CHANGELOG.md updated with a link to the PR (not the Issue)
Rebased/mergeable
Tests pass
Sign CLA (if not already signed)

ebb-tide

Everything was amazing and nothing hurt.

ebb-tide · 2018-05-10T16:56:58Z

ui/src/shared/apis/index.js

-  })
+  }
+
+  return AJAX(params)


- Fixes Pagerduty save button

bthesorceror added 3 commits May 10, 2018 00:24

Provides checkbox for disabling kapacitor services

892b137

Only enable test button when config is enabled

cb54fd0

Update changelog

b24ca2d

ebb-tide self-requested a review May 10, 2018 16:48

ebb-tide approved these changes May 10, 2018

View reviewed changes

ui/src/shared/apis/index.js

})

}

return AJAX(params)

Copy link

Contributor

ebb-tide May 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Enhance messaging for disabled configuration

3768b63

- Fixes Pagerduty save button

bthesorceror merged commit 2126c50 into master May 10, 2018

bthesorceror deleted the features/disable-alert-configs branch May 10, 2018 18:27

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Features/disable alert configs #3416

Features/disable alert configs #3416

bthesorceror commented May 10, 2018 •

edited

Loading

ebb-tide left a comment

ebb-tide May 10, 2018

Features/disable alert configs #3416

Features/disable alert configs #3416

Conversation

bthesorceror commented May 10, 2018 • edited Loading

ebb-tide left a comment

Choose a reason for hiding this comment

ebb-tide May 10, 2018

Choose a reason for hiding this comment

bthesorceror commented May 10, 2018 •

edited

Loading