From 4dac2f51f7d21f773938d256b85cdf0a5bcc61fa Mon Sep 17 00:00:00 2001 From: Michael Desa Date: Mon, 9 Sep 2019 20:42:53 -0400 Subject: [PATCH] fix(ui): use slack as default endpoint This is so that in cloud there is not a weird UX for initial users who see http, but cannot create an http endpoint --- ui/cypress/e2e/notificationEndpoints.test.ts | 2 +- ui/src/alerting/constants/index.ts | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/ui/cypress/e2e/notificationEndpoints.test.ts b/ui/cypress/e2e/notificationEndpoints.test.ts index e626445e38..c45357fb79 100644 --- a/ui/cypress/e2e/notificationEndpoints.test.ts +++ b/ui/cypress/e2e/notificationEndpoints.test.ts @@ -50,7 +50,7 @@ describe('Notification Endpoints', () => { .click() .within(() => { cy.getByTestID('endpoint--dropdown--button').within(() => { - cy.contains('HTTP') + cy.contains('Slack') }) cy.getByTestID('endpoint--dropdown-item pagerduty').click() diff --git a/ui/src/alerting/constants/index.ts b/ui/src/alerting/constants/index.ts index 9bb32c33ab..37bcd6e4ff 100644 --- a/ui/src/alerting/constants/index.ts +++ b/ui/src/alerting/constants/index.ts @@ -146,13 +146,11 @@ export const DEFAULT_ENDPOINT_URLS = { export const NEW_ENDPOINT_DRAFT: NotificationEndpoint = { name: 'Name this Endpoint', - method: 'POST', - authMethod: 'none', description: '', status: 'active', - type: 'http', + type: 'slack', token: '', - url: DEFAULT_ENDPOINT_URLS['http'], + url: DEFAULT_ENDPOINT_URLS['slack'], } export const NEW_ENDPOINT_FIXTURES: NotificationEndpoint[] = [