Adding PagerDuty icon to connectors cards (elastic#60805)

* adding PagerDuty icon to connectors cards * Fix jest * remove unnecessary global typings from canvas and sync global typings in xpack with oss Co-authored-by: Andrea Del Rio <[email protected]> Co-authored-by: spalger <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
mikecote · Mar 31, 2020 · 0cceb14 · 0cceb14
1 parent 886979b
commit 0cceb14
Show file tree

Hide file tree

Showing 7 changed files with 54 additions and 17 deletions.
diff --git a/examples/alerting_example/tsconfig.json b/examples/alerting_example/tsconfig.json
@@ -1,9 +1,7 @@
 {
   "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "outDir": "./target",
-    "skipLibCheck": true,
-    "resolveJsonModule": true
+    "outDir": "./target"
   },
   "include": [
     "index.ts",

diff --git a/typings/index.d.ts b/typings/index.d.ts
@@ -23,6 +23,27 @@ declare module '*.html' {
   export default template;
 }
 
+declare module '*.png' {
+  const content: string;
+  // eslint-disable-next-line import/no-default-export
+  export default content;
+}
+
+declare module '*.svg' {
+  const content: string;
+  // eslint-disable-next-line import/no-default-export
+  export default content;
+}
+
+// allow JSON files to be imported directly without lint errors
+// see: https://github.com/palantir/tslint/issues/1264#issuecomment-228433367
+// and: https://github.com/Microsoft/TypeScript/wiki/Breaking-Changes#arbitrary-expressions-are-forbidden-in-export-assignments-in-ambient-contexts
+declare module '*.json' {
+  const json: any;
+  // eslint-disable-next-line import/no-default-export
+  export default json;
+}
+
 type MethodKeysOf<T> = {
   [K in keyof T]: T[K] extends (...args: any[]) => any ? K : never;
 }[keyof T];
@@ -37,3 +58,7 @@ type DeeplyMockedKeys<T> = {
     : DeeplyMockedKeys<T[P]>;
 } &
   T;
+
+type Writable<T> = {
+  -readonly [K in keyof T]: T[K];
+};
diff --git a/x-pack/legacy/plugins/canvas/types/webpack.d.ts b/x-pack/legacy/plugins/canvas/types/webpack.d.ts
diff --git a/...ers_actions_ui/public/application/components/builtin_action_types/pagerduty.svg b/...ers_actions_ui/public/application/components/builtin_action_types/pagerduty.svg
diff --git a/...triggers_actions_ui/public/application/components/builtin_action_types/pagerduty.test.tsx b/...triggers_actions_ui/public/application/components/builtin_action_types/pagerduty.test.tsx
@@ -30,7 +30,7 @@ beforeAll(() => {
 describe('actionTypeRegistry.get() works', () => {
   test('action type static data is as expected', () => {
     expect(actionTypeModel.id).toEqual(ACTION_TYPE_ID);
-    expect(actionTypeModel.iconClass).toEqual('apps');
+    expect(actionTypeModel.iconClass).toEqual('test-file-stub');
   });
 });
 

diff --git a/...gins/triggers_actions_ui/public/application/components/builtin_action_types/pagerduty.tsx b/...gins/triggers_actions_ui/public/application/components/builtin_action_types/pagerduty.tsx
@@ -25,11 +25,12 @@ import {
   ActionParamsProps,
 } from '../../../types';
 import { PagerDutyActionParams, PagerDutyActionConnector } from './types';
+import pagerDutySvg from './pagerduty.svg';
 
 export function getActionType(): ActionTypeModel {
   return {
     id: '.pagerduty',
-    iconClass: 'apps',
+    iconClass: pagerDutySvg,
     selectMessage: i18n.translate(
       'xpack.triggersActionsUI.components.builtinActionTypes.pagerDutyAction.selectMessageText',
       {

diff --git a/x-pack/typings/index.d.ts b/x-pack/typings/index.d.ts
@@ -10,6 +10,18 @@ declare module '*.html' {
   export default template;
 }
 
+declare module '*.png' {
+  const content: string;
+  // eslint-disable-next-line import/no-default-export
+  export default content;
+}
+
+declare module '*.svg' {
+  const content: string;
+  // eslint-disable-next-line import/no-default-export
+  export default content;
+}
+
 declare module 'lodash/internal/toPath' {
   function toPath(value: string | string[]): string[];
   export = toPath;