Skip to content

Commit

Permalink
Enable the roles tab in shared apps from view mode
Browse files Browse the repository at this point in the history
  • Loading branch information
ShanChathusanda93 committed Feb 9, 2025
1 parent 4c6ffe0 commit b041a81
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
6 changes: 6 additions & 0 deletions .changeset/lovely-items-protect.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
"@wso2is/admin.extensions.v1": patch
"@wso2is/admin.roles.v2": patch
---

enble roles tab for shared apps
4 changes: 1 addition & 3 deletions features/admin.extensions.v1/configs/application.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -277,8 +277,6 @@ export const applicationConfig: ApplicationConfig = {

// Enable the roles tab for supported templates when the api resources config is enabled.
if (apiResourceFeatureEnabled
&& (!application?.advancedConfigurations?.fragment || window["AppUtils"].getConfig().ui.features?.
applicationRoles?.enabled)
&& (
application?.advancedConfigurations?.fragment ||
(application?.templateId === ApplicationManagementConstants.CUSTOM_APPLICATION_OIDC
Expand All @@ -302,7 +300,7 @@ export const applicationConfig: ApplicationConfig = {
<ResourceTab.Pane controlledSegmentation>
<ApplicationRoles
onUpdate={ onApplicationUpdate }
readOnly={ isReadOnly }
readOnly={ isReadOnly || application?.advancedConfigurations?.fragment }
/>
</ResourceTab.Pane>
)
Expand Down
12 changes: 6 additions & 6 deletions features/admin.roles.v2/components/application-roles.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,9 @@ import { useRequiredScopes } from "@wso2is/access-control";
import { updateApplicationDetails } from "@wso2is/admin.applications.v1/api/application";
import { useGetApplication } from "@wso2is/admin.applications.v1/api/use-get-application";
import { ApplicationInterface } from "@wso2is/admin.applications.v1/models/application";
import { OrganizationType } from "@wso2is/admin.core.v1/constants/organization-constants";
import { history } from "@wso2is/admin.core.v1/helpers/history";
import { FeatureConfigInterface } from "@wso2is/admin.core.v1/models/config";
import { AppState } from "@wso2is/admin.core.v1/store";
import { useGetCurrentOrganizationType } from "@wso2is/admin.organizations.v1/hooks/use-get-organization-type";
import { AlertLevels, IdentifiableComponentInterface } from "@wso2is/core/models";
import { addAlert } from "@wso2is/core/store";
import {
Expand Down Expand Up @@ -101,7 +99,6 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
const { t } = useTranslation();
const dispatch: Dispatch<any> = useDispatch();
const { getLink } = useDocumentation();
const { organizationType } = useGetCurrentOrganizationType();
const { data: application } = useGetApplication(appId, !!appId);

const [ isLoading, setIsLoading ] = useState<boolean>(false);
Expand All @@ -120,7 +117,6 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
useState<BasicRoleInterface[]>(application?.associatedRoles?.roles ?? []);
const [ activeOption, setActiveOption ] = useState<BasicRoleInterface>(undefined);

const isReadOnly: boolean = readOnly || organizationType === OrganizationType.SUBORGANIZATION;
const [ showWizard, setShowWizard ] = useState<boolean>(false);

const featureConfig: FeatureConfigInterface = useSelector((state: AppState) => state?.config?.ui?.features);
Expand Down Expand Up @@ -334,10 +330,12 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
"rolesV2.application")
}
data-componentid={ `${ componentId }-application-audience-checkbox` }
disabled={ readOnly }
/>
</Grid.Column>
<Grid.Column width={ 6 }>
{
!readOnly &&
roleAudience === RoleAudienceTypes.APPLICATION &&
hasRoleCreatePermissions
&& (
Expand Down Expand Up @@ -368,6 +366,7 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
"rolesV2.organization")
}
data-componentid={ `${ componentId }-organization-audience-checkbox` }
disabled={ readOnly }
/>
</Grid.Column>
</Grid.Row>
Expand Down Expand Up @@ -400,6 +399,7 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
<Autocomplete
multiple
disableCloseOnSelect
readOnly={ readOnly }
loading={ isLoading }
options={ roleList }
value={ selectedRoles ?? [] }
Expand All @@ -410,7 +410,7 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
renderInput={ (params: AutocompleteRenderInputParams) => (
<TextField
{ ...params }
placeholder={ !isReadOnly && t("extensions:develop.applications.edit." +
placeholder={ !readOnly && t("extensions:develop.applications.edit." +
"sections.rolesV2.searchPlaceholder") }
/>
) }
Expand Down Expand Up @@ -462,7 +462,7 @@ export const ApplicationRoles: FunctionComponent<ApplicationRolesSettingsInterfa
</Grid.Row>
) : null }
{
(
!readOnly && (
<Grid.Row className="mt-4">
<Grid.Column width={ 16 }>
<PrimaryButton
Expand Down

0 comments on commit b041a81

Please sign in to comment.