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

Application warning banner #6813

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all 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
7 changes: 7 additions & 0 deletions .changeset/tough-mugs-complain.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
"@wso2is/admin.applications.v1": patch
"@wso2is/console": patch
"@wso2is/i18n": patch
---

Application warning banner.
2 changes: 2 additions & 0 deletions features/admin.applications.v1/models/application-inbound.ts
Original file line number Diff line number Diff line change
Expand Up @@ -193,6 +193,8 @@ export interface OIDCDataInterface {
subject?: SubjectConfigInterface;
isFAPIApplication?: boolean;
hybridFlow?: HybridFlowConfigurationInterface;
useClientIdAsSubClaimForAppTokens?: boolean;
omitUsernameInIntrospectionRespForAppTokens?: boolean;
}

/**
Expand Down
26 changes: 26 additions & 0 deletions features/admin.applications.v1/pages/application-edit.scss
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,29 @@
.application-branding-link {
cursor: pointer;
}

.ignore-once-button {
color: #788997;
}

.banner-detail-card {
border: 0;
border-top-left-radius: 0;
border-top-right-radius: 0;
background: #fffaf3;
padding: 5px;
padding-left: 35px;
}

.application-outdated-alert-expanded-view {
border-bottom-left-radius: 0;
border-bottom-right-radius: 0;
}

.spaced-list li {
margin-bottom: 10px;
}

.banner-wrapper {
margin-bottom: 20px;
}
410 changes: 408 additions & 2 deletions features/admin.applications.v1/pages/application-edit.tsx

Large diffs are not rendered by default.

28 changes: 28 additions & 0 deletions modules/i18n/src/models/namespaces/applications-ns.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1377,6 +1377,34 @@ export interface ApplicationsNS {
};
};
};
legacyApplicationTokens: {
heading: string;
alert : {
title: string;
content: string;
viewButton: string;
cancelButton: string;
}
confirmationModal: {
assertionHint: string;
header: string;
message: string;
content: string;
},
fields: {
commonInstruction: string;
useClientIdAsSubClaimForAppTokens: {
instruction: string;
label: string;
hint: string;
};
omitUsernameInIntrospectionRespForAppTokens: {
instruction: string;
label: string;
hint: string;
};
}
};
logoutURLs: {
heading: string;
fields: {
Expand Down
33 changes: 33 additions & 0 deletions modules/i18n/src/translations/en-US/portals/applications.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1643,6 +1643,39 @@ export const applications: ApplicationsNS = {
},
heading: "ID Token"
},
legacyApplicationTokens: {
heading: "Legacy Application Tokens",
alert : {
title: "Application is outdated.",
content: "This application is using an outdated behavior for application tokens.",
viewButton: "View Details",
cancelButton: "Ignore Once"
},
confirmationModal: {
header: "Have you done the relevant changes?",
message: "Proceeding the action without making relevant change will cause the client application behavior break.",
content: "Confirming the action will,",
assertionHint: "Please confirm your action"
},
fields: {
commonInstruction: "Change the customer-end applications accordingly to recieve the below updates.",
useClientIdAsSubClaimForAppTokens: {
instruction: "Application access token sub attribute will be client_id generated for an application.",
label: "Set client_id as the sub attribute value for Application tokens",
hint: "For application tokens, the sub attribute was previosuly set to the "
+ "application owner's user_id. However, to support a more industry standard "
+ "solution, this value will be changed to the client ID for application tokens."
},
omitUsernameInIntrospectionRespForAppTokens: {
instruction: "Introspection response for application access token will not include the username attribute.",
label: "Omit sending username attribute in the Introspection response for Application tokens",
hint: "For access tokens, the previous behavior includes sending the username attribute"
+ " in the introspection response. However, to support a more industry standard"
+ " solution, the introspection response for application tokens will no longer"
+ " include the username attribute."
}
}
},
logoutURLs: {
fields: {
back: {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020, WSO2 LLC. (https://www.wso2.com). All Rights Reserved.
* Copyright (c) 2020, WSO2 LLC. (https://www.wso2.com).
*
* WSO2 LLC. licenses this file to you under the Apache License,
* Version 2.0 (the "License"); you may not use this file except
Expand Down Expand Up @@ -41,6 +41,10 @@ export interface PageHeaderPropsInterface extends LoadableComponentInterface, Te
* Column width for the action container grid.
*/
actionColumnWidth?: SemanticWIDTHS;
/**
* Alert component displayed in the page header.
*/
alertBanner?: ReactNode;
/**
* Go back button.
*/
Expand Down Expand Up @@ -127,6 +131,7 @@ export const PageHeader: React.FunctionComponent<PageHeaderPropsInterface> = (
const {
action,
actionColumnWidth,
alertBanner,
backButton,
bottomMargin,
className,
Expand Down Expand Up @@ -301,6 +306,9 @@ export const PageHeader: React.FunctionComponent<PageHeaderPropsInterface> = (
)
)
}
{
alertBanner
}
{
action
? (
Expand Down
Loading