Skip to content

Commit

Permalink
Deangular: removes angular from notify and utils of opendashboards le…
Browse files Browse the repository at this point in the history
…gacy (#5085)

* Basic working version of OSD without angular (#4982)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* deangularized notify and utils folders in the opendashboards legacy directory

Signed-off-by: Cameron Reed <chyltonreed@gmail.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Cameron Reed <chyltonreed@gmail.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Cameron Reed <chyltonreed@gmail.com>
  • Loading branch information
3 people authored Sep 28, 2023
1 parent 5fe2fae commit 61f40ea
Show file tree
Hide file tree
Showing 11 changed files with 11 additions and 566 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -466,4 +466,4 @@
"node": ">=14.20.1 <19",
"yarn": "^1.22.10"
}
}
}
1 change: 0 additions & 1 deletion src/plugins/opensearch_dashboards_legacy/public/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,3 @@ export const plugin = (initializerContext: PluginInitializerContext) =>
export * from './plugin';

export * from './notify';
export * from './utils';
Original file line number Diff line number Diff line change
Expand Up @@ -29,21 +29,15 @@
*/

import { FatalErrorsSetup } from '../../../../../core/public';
import {
AngularHttpError,
formatAngularHttpError,
isAngularHttpError,
} from './format_angular_http_error';

/**
* Adds an error to the list of fatal errors.
* @deprecated Use `core.fatalErrors.add` instead
*/
export function addFatalError(
fatalErrors: FatalErrorsSetup,
error: AngularHttpError | Error | string,
location?: string
) {
// add support for angular http errors to newPlatformFatalErrors
if (isAngularHttpError(error)) {
error = formatAngularHttpError(error);
}

fatalErrors.add(error, location);
}
fatalErrors: FatalErrorsSetup,

Check failure on line 38 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`

Check failure on line 38 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`
error: Error | string,

Check failure on line 39 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`

Check failure on line 39 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`
location?: string

Check failure on line 40 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`

Check failure on line 40 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`
) {

Check failure on line 41 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`

Check failure on line 41 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`
fatalErrors.add(error, location);

Check failure on line 42 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`

Check failure on line 42 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Delete `··`
}

Check failure on line 43 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Replace `··}` with `}⏎`

Check failure on line 43 in src/plugins/opensearch_dashboards_legacy/public/notify/lib/add_fatal_error.ts

View workflow job for this annotation

GitHub Actions / Build and Verify on Linux (ciGroup1)

Replace `··}` with `}⏎`

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,3 @@
export { formatOpenSearchMsg } from './format_opensearch_msg';
export { formatMsg } from './format_msg';
export { formatStack } from './format_stack';
export {
isAngularHttpError,
formatAngularHttpError,
AngularHttpError,
} from './format_angular_http_error';
export { addFatalError } from './add_fatal_error';
37 changes: 0 additions & 37 deletions src/plugins/opensearch_dashboards_legacy/public/utils/index.ts

This file was deleted.

This file was deleted.

31 changes: 0 additions & 31 deletions src/plugins/opensearch_dashboards_legacy/public/utils/private.d.ts

This file was deleted.

Loading

0 comments on commit 61f40ea

Please sign in to comment.