Skip to content
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

Re-add custom flash service so engines can extend cluster's template #19963

Merged
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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'],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for doing this!

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
5 changes: 1 addition & 4 deletions ui/app/components/transform-role-edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,7 @@ export default TransformBase.extend({
message =
'This role was edited to remove transformations, but this role was not removed from those transformations’ allowed_roles due to a lack of permissions.';
}
this.flashMessages.info(message, {
sticky: true,
priority: 300,
});
this.flashMessages.stickyInfo(message);
}
});
},
Expand Down
5 changes: 1 addition & 4 deletions ui/app/components/transformation-edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,10 +82,7 @@ export default TransformBase.extend({
// eg. trying to update a role with empty array as transformations
message = `You've edited the allowed_roles for this transformation. However, the corresponding edits to some roles' transformations were not made`;
}
this.flashMessages.info(message, {
sticky: true,
priority: 300,
});
this.flashMessages.stickyInfo(message);
}
});
},
Expand Down
5 changes: 1 addition & 4 deletions ui/app/routes/vault/cluster/auth.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,7 @@ export default ClusterRouteBase.extend({

afterModel() {
if (config.welcomeMessage) {
this.flashMessages.info(config.welcomeMessage, {
sticky: true,
priority: 300,
});
this.flashMessages.stickyInfo(config.welcomeMessage);
}
},
});
15 changes: 15 additions & 0 deletions ui/app/services/flash-messages.ts
hellobontempo marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Contributor Author

@hellobontempo hellobontempo Apr 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

original original file: https://github.com/hashicorp/vault/pull/19925/files#diff-5ebd44ba5909c95c331fd105e463a0332fa06bfc2f1244a4324d2518c29d9e59

(still not using stickyInfo anywhere)

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

export default class FlashMessageService extends FlashMessages {
  stickyInfo(message: string) {
    return this.info(message, {
      sticky: true,
      priority: 300,
    });
  }
}

Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
/**
* Copyright (c) HashiCorp, Inc.
* SPDX-License-Identifier: MPL-2.0
*/

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

export default class FlashMessages extends EmberCliFlash {
stickyInfo(message: string) {
return this.info(message, {
sticky: true,
priority: 300,
});
}
}
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'],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opted to make syntax the same across the app

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
4 changes: 2 additions & 2 deletions ui/lib/pki/addon/components/page/pki-certificate-details.ts
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 FlashMessages from 'vault/services/flash-messages';
import DownloadService from 'vault/services/download';
import PkiCertificateBaseModel from 'vault/models/pki/certificate/base';

Expand All @@ -20,7 +20,7 @@ interface Args {
}

export default class PkiCertificateDetailsComponent extends Component<Args> {
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@service declare readonly download: DownloadService;

@action
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import PkiActionModel from 'vault/models/pki/action';
import { Breadcrumb } from 'vault/vault/app-types';

Expand All @@ -29,7 +29,7 @@ interface Args {
export default class PkiConfigureCreate extends Component<Args> {
@service declare readonly store: Store;
@service declare readonly router: Router;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;

@tracked title = 'Configure PKI';

Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import PkiIssuerModel from 'vault/models/pki/issuer';

interface Args {
Expand All @@ -20,7 +20,7 @@ interface Args {

export default class PkiIssuerEditComponent extends Component<Args> {
@service declare readonly router: RouterService;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;

@tracked usageValues: Array<string> = [];
@tracked error = null;
Expand Down
4 changes: 2 additions & 2 deletions ui/lib/pki/addon/components/page/pki-issuer-rotate-root.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import errorMessage from 'vault/utils/error-message';
// TYPES
import Store from '@ember-data/store';
import Router from '@ember/routing/router';
import FlashMessageService from 'vault/services/flash-messages';
import FlashMessages from 'vault/services/flash-messages';
import SecretMountPath from 'vault/services/secret-mount-path';
import PkiIssuerModel from 'vault/models/pki/issuer';
import PkiActionModel from 'vault/vault/models/pki/action';
Expand All @@ -30,7 +30,7 @@ const RADIO_BUTTON_KEY = {
export default class PagePkiIssuerRotateRootComponent extends Component<Args> {
@service declare readonly store: Store;
@service declare readonly router: Router;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@service declare readonly secretMountPath: SecretMountPath;

@tracked displayedForm = RADIO_BUTTON_KEY.oldSettings;
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages 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 All @@ -16,7 +16,7 @@ interface Args {

export default class PkiKeyDetails extends Component<Args> {
@service declare readonly router: RouterService;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;

@action
async deleteKey() {
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages 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 All @@ -18,7 +18,7 @@ interface Args {

export default class DetailsPage extends Component<Args> {
@service declare readonly router: RouterService;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@service declare readonly secretMountPath: SecretMountPath;

get breadcrumbs() {
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import PkiActionModel from 'vault/models/pki/action';
import errorMessage from 'vault/utils/error-message';

Expand Down Expand Up @@ -42,7 +42,7 @@ interface Args {
* @param {Object} adapterOptions - object passed as adapterOptions on the model.save method
*/
export default class PkiGenerateCsrComponent extends Component<Args> {
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;

@tracked modelValidations = null;
@tracked error: string | null = null;
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import errorMessage from 'vault/utils/error-message';
import { parsedParameters } from 'vault/utils/parse-pki-cert-oids';

Expand Down Expand Up @@ -50,7 +50,7 @@ interface Args {
* @param {Object} adapterOptions - object passed as adapterOptions on the model.save method
*/
export default class PkiGenerateRootComponent extends Component<Args> {
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@service declare readonly router: RouterService;

@tracked modelValidations = null;
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import { inject as service } from '@ember/service';
import { task } from 'ember-concurrency';
import { tracked } from '@glimmer/tracking';
Expand Down Expand Up @@ -41,7 +41,7 @@ interface Args {
}

export default class PkiImportPemBundle extends Component<Args> {
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;

@tracked errorBanner = '';

Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages 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 All @@ -25,7 +25,7 @@ interface Args {
export default class PkiRoleGenerate extends Component<Args> {
@service declare readonly router: Router;
@service declare readonly store: Store;
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@service declare readonly download: DownloadService;

@tracked errorBanner = '';
Expand Down
4 changes: 2 additions & 2 deletions 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 FlashMessages from 'vault/services/flash-messages';
import errorMessage from 'vault/utils/error-message';

interface Args {
Expand All @@ -19,7 +19,7 @@ interface Args {
}

export default class PkiSignIntermediateFormComponent extends Component<Args> {
@service declare readonly flashMessages: FlashMessageService;
@service declare readonly flashMessages: FlashMessages;
@tracked errorBanner = '';
@tracked inlineFormAlert = '';
@tracked modelValidations = null;
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
12 changes: 6 additions & 6 deletions ui/types/ember-cli-flash/services/flash-messages.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,10 @@ declare module 'ember-cli-flash/services/flash-messages' {
}

interface FlashFunction {
(message: string, options?: Partial<MessageOptions>): FlashMessageService;
(message: string, options?: Partial<MessageOptions>): FlashMessages;
}

class FlashMessageService extends Service {
class FlashMessages extends Service {
queue: A<FlashObject>;
success: FlashFunction;
warning: FlashFunction;
Expand All @@ -39,11 +39,11 @@ declare module 'ember-cli-flash/services/flash-messages' {
danger: FlashFunction;
alert: FlashFunction;
secondary: FlashFunction;
add(messageInfo: CustomMessageInfo): FlashMessageService;
clearMessages(): FlashMessageService;
registerTypes(types: string[]): FlashMessageService;
add(messageInfo: CustomMessageInfo): FlashMessages;
clearMessages(): FlashMessages;
registerTypes(types: string[]): FlashMessages;
getFlashObject(): FlashObject;
}

export default FlashMessageService;
export default FlashMessages;
}