-
Notifications
You must be signed in to change notification settings - Fork 256
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add OpsGenie v2 to UI #3369
Add OpsGenie v2 to UI #3369
Conversation
…when submitting form
const {configSections} = this.state | ||
const {hash} = this.props | ||
|
||
// console.log(configSections) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you mind removing this? 😄
@@ -345,7 +376,7 @@ class AlertTabs extends Component { | |||
<TabList customClass="config-endpoint--tabs"> | |||
{_.reduce( | |||
configSections, | |||
(acc, _cur, k) => { | |||
(acc, __, k) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work, one console log to remove but other than that looks good.
I'm going to merge in order to not have conflicts with my current branch, which will use the TypeScript work here, and let @ischolten update swagger for the new OpsGenie2 config upon her return: #3382. |
Swagger was updated by PR #3398 |
Closes #2929
Briefly describe your proposed changes:
Add config in UI for OpsGenie v2
Update kapacitor dependency to 1.5 rc 2