Skip to content

Commit

Permalink
Re-add custom flash service so engines can extend cluster's template (h…
Browse files Browse the repository at this point in the history
…ashicorp#19963)

* Revert "UI: Remove custom service (hashicorp#19925)"

This reverts commit b04751d.

* replace stickyInfo with options info

* revert replacing custom stickyInfo

* change flash message name to be consistent throughout application

* make service imports consistent for k8 engine

* replace stickyInfo with options info

* Revert "change flash message name to be consistent throughout application"

This reverts commit 17de498.

* add comment

---------

Co-authored-by: Chelsea Shaw <cshaw@hashicorp.com>
  • Loading branch information
hellobontempo and hashishaw committed Apr 3, 2023
1 parent 33731d6 commit e973716
Show file tree
Hide file tree
Showing 18 changed files with 29 additions and 19 deletions.
8 changes: 5 additions & 3 deletions ui/app/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ export default class App extends Application {
engines = {
openApiExplorer: {
dependencies: {
services: ['auth', 'namespace', 'router', 'version'],
services: ['auth', 'flash-messages', 'namespace', 'router', 'version'],
},
},
replication: {
dependencies: {
services: ['auth', 'namespace', 'replication-mode', 'router', 'store', 'version'],
services: ['auth', 'flash-messages', 'namespace', 'replication-mode', 'router', 'store', 'version'],
externalRoutes: {
replication: 'vault.cluster.replication.index',
},
Expand All @@ -31,6 +31,7 @@ export default class App extends Application {
services: [
'auth',
'download',
'flash-messages',
'namespace',
'path-help',
'router',
Expand All @@ -45,7 +46,7 @@ export default class App extends Application {
},
kubernetes: {
dependencies: {
services: ['router', 'store', 'secret-mount-path'],
services: ['router', 'store', 'secret-mount-path', 'flash-messages'],
externalRoutes: {
secrets: 'vault.cluster.secrets.backends',
},
Expand All @@ -56,6 +57,7 @@ export default class App extends Application {
services: [
'auth',
'download',
'flash-messages',
'namespace',
'path-help',
'router',
Expand Down
13 changes: 13 additions & 0 deletions ui/app/services/flash-messages.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
/**
* Copyright (c) HashiCorp, Inc.
* SPDX-License-Identifier: MPL-2.0
*/

import FlashMessages from 'ember-cli-flash/services/flash-messages';

/*
we extend the ember-cli-flash service here so each ember engine can
import 'flash-messages' as a dependency giving it access to the
<FlashMessage> template in the main app's cluster.hbs file
*/
export default class FlashMessageService extends FlashMessages {}
1 change: 0 additions & 1 deletion ui/lib/kmip/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
],
"dependencies": {
"ember-cli-htmlbars": "*",
"ember-cli-flash": "*",
"ember-cli-babel": "*"
},
"ember-addon": {
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/kubernetes/addon/engine.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export default class KubernetesEngine extends Engine {
modulePrefix = modulePrefix;
Resolver = Resolver;
dependencies = {
services: ['router', 'store', 'secret-mount-path', 'flashMessages'],
services: ['router', 'store', 'secret-mount-path', 'flash-messages'],
externalRoutes: ['secrets'],
};
}
Expand Down
1 change: 0 additions & 1 deletion ui/lib/kubernetes/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
"dependencies": {
"ember-cli-htmlbars": "*",
"ember-cli-babel": "*",
"ember-cli-flash": "*",
"ember-concurrency": "*",
"@ember/test-waiters": "*",
"ember-inflector": "*"
Expand Down
1 change: 0 additions & 1 deletion ui/lib/open-api-explorer/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
"dependencies": {
"ember-cli-htmlbars": "*",
"ember-cli-babel": "*",
"ember-cli-flash": "*",
"ember-auto-import": "*",
"swagger-ui-dist": "*"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { action } from '@ember/object';
import { task } from 'ember-concurrency';
import { waitFor } from '@ember/test-waiters';
import errorMessage from 'vault/utils/error-message';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import DownloadService from 'vault/services/download';
import PkiCertificateBaseModel from 'vault/models/pki/certificate/base';

Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/page/pki-configure-create.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { tracked } from '@glimmer/tracking';
// TYPES
import Store from '@ember-data/store';
import Router from '@ember/routing/router';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import PkiActionModel from 'vault/models/pki/action';
import { Breadcrumb } from 'vault/vault/app-types';

Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/page/pki-issuer-edit.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import { task } from 'ember-concurrency';
import { waitFor } from '@ember/test-waiters';
import errorMessage from 'vault/utils/error-message';
import RouterService from '@ember/routing/router-service';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import PkiIssuerModel from 'vault/models/pki/issuer';

interface Args {
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/page/pki-key-details.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
import { action } from '@ember/object';
import Component from '@glimmer/component';
import RouterService from '@ember/routing/router-service';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import { inject as service } from '@ember/service';
import errorMessage from 'vault/utils/error-message';
import PkiKeyModel from 'vault/models/pki/key';
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/page/pki-role-details.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
import { action } from '@ember/object';
import RouterService from '@ember/routing/router-service';
import Component from '@glimmer/component';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import SecretMountPath from 'vault/services/secret-mount-path';
import { inject as service } from '@ember/service';
import errorMessage from 'vault/utils/error-message';
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/pki-generate-csr.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import { action } from '@ember/object';
import { task } from 'ember-concurrency';
import { waitFor } from '@ember/test-waiters';
import { expandAttributeMeta } from 'vault/utils/field-to-attrs';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import PkiActionModel from 'vault/models/pki/action';
import errorMessage from 'vault/utils/error-message';

Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/pki-generate-root.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import { tracked } from '@glimmer/tracking';
import { task } from 'ember-concurrency';
import PkiActionModel from 'vault/models/pki/action';
import PkiUrlsModel from 'vault/models/pki/urls';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import errorMessage from 'vault/utils/error-message';
import { parsedParameters } from 'vault/utils/parse-pki-cert-oids';

Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/pki-import-pem-bundle.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import { action } from '@ember/object';
import Component from '@glimmer/component';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import { inject as service } from '@ember/service';
import { task } from 'ember-concurrency';
import { tracked } from '@glimmer/tracking';
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/pki-role-generate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import Router from '@ember/routing/router';
import Store from '@ember-data/store';
import { tracked } from '@glimmer/tracking';
import errorMessage from 'vault/utils/error-message';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import DownloadService from 'vault/services/download';
import PkiCertificateGenerateModel from 'vault/models/pki/certificate/generate';
import PkiCertificateSignModel from 'vault/models/pki/certificate/sign';
Expand Down
2 changes: 1 addition & 1 deletion ui/lib/pki/addon/components/pki-sign-intermediate-form.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import Component from '@glimmer/component';
import { tracked } from '@glimmer/tracking';
import { task } from 'ember-concurrency';
import PkiCertificateSignIntermediate from 'vault/models/pki/certificate/sign-intermediate';
import FlashMessageService from 'ember-cli-flash/services/flash-messages';
import FlashMessageService from 'vault/services/flash-messages';
import errorMessage from 'vault/utils/error-message';

interface Args {
Expand Down
1 change: 0 additions & 1 deletion ui/lib/pki/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
"dependencies": {
"ember-cli-babel": "*",
"ember-cli-htmlbars": "*",
"ember-cli-flash": "*",
"ember-cli-typescript": "*",
"@types/ember": "latest",
"@types/ember-data": "latest",
Expand Down
1 change: 0 additions & 1 deletion ui/lib/replication/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
],
"dependencies": {
"ember-cli-htmlbars": "*",
"ember-cli-flash": "*",
"ember-cli-babel": "*"
},
"ember-addon": {
Expand Down

0 comments on commit e973716

Please sign in to comment.