Skip to content

Commit

Permalink
Merge branch 'main' into fix/1066-hide-technicalUserManagement-button
Browse files Browse the repository at this point in the history
  • Loading branch information
oyo authored Sep 6, 2024
2 parents cfb8553 + 4ac2435 commit a0001d4
Show file tree
Hide file tree
Showing 37 changed files with 822 additions and 220 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:

# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v2.227
uses: github/codeql-action/init@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v2.227
with:
languages: ${{ matrix.language }}
# If you wish to specify custom queries, you can do so here or in a config file.
Expand All @@ -89,7 +89,7 @@ jobs:
# Automates dependency installation for Python, Ruby, and JavaScript, optimizing the CodeQL analysis setup.
# If this step fails, then you should remove it and run the build manually (see below)
- name: Autobuild
uses: github/codeql-action/autobuild@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v2.227
uses: github/codeql-action/autobuild@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v2.227

# ℹ️ Command-line programs to run using the OS shell.
# 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
Expand All @@ -102,6 +102,6 @@ jobs:
# ./location_of_script_within_repo/buildscript.sh

- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v2.227
uses: github/codeql-action/analyze@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v2.227
with:
category: '/language:${{matrix.language}}'
2 changes: 1 addition & 1 deletion .github/workflows/dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:
if: steps.dependencies-changed.outputs.changed == 'true'

- name: Upload DEPENDENCIES file
uses: actions/upload-artifact@834a144ee995460fba8ed112a2fc961b36a5ec5a # v4.3.6
uses: actions/upload-artifact@50769540e7f4bd5e21e526ee35c689e35e0d6874 # v4.4.0
with:
path: DEPENDENCIES
if: steps.dependencies-changed.outputs.changed == 'true'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/kics.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,6 @@ jobs:
# Upload findings to GitHub Advanced Security Dashboard
- name: Upload SARIF file for GitHub Advanced Security Dashboard
if: always()
uses: github/codeql-action/upload-sarif@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v3.26.5
uses: github/codeql-action/upload-sarif@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v3.26.6
with:
sarif_file: kicsResults/results.sarif
4 changes: 2 additions & 2 deletions .github/workflows/trivy-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ jobs:
vuln-type: 'os,library'

- name: Upload Trivy scan results to GitHub Security tab
uses: github/codeql-action/upload-sarif@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v3.26.5
uses: github/codeql-action/upload-sarif@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v3.26.6
if: always()
with:
sarif_file: 'trivy-results1.sarif'
Expand Down Expand Up @@ -96,6 +96,6 @@ jobs:

- name: Upload Trivy scan results to GitHub Security tab
if: always()
uses: github/codeql-action/upload-sarif@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v3.26.5
uses: github/codeql-action/upload-sarif@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v3.26.6
with:
sarif_file: 'trivy-results2.sarif'
4 changes: 2 additions & 2 deletions .github/workflows/trivy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ jobs:
vuln-type: 'os,library'

- name: Upload Trivy scan results to GitHub Security tab
uses: github/codeql-action/upload-sarif@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v3.26.5
uses: github/codeql-action/upload-sarif@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v3.26.6
if: always()
with:
sarif_file: 'trivy-results1.sarif'
Expand Down Expand Up @@ -96,6 +96,6 @@ jobs:

- name: Upload Trivy scan results to GitHub Security tab
if: always()
uses: github/codeql-action/upload-sarif@2c779ab0d087cd7fe7b826087247c2c81f27bfa6 # v3.26.5
uses: github/codeql-action/upload-sarif@4dd16135b69a43b6c8efb853346f8437d92d3c93 # v3.26.6
with:
sarif_file: 'trivy-results2.sarif'
14 changes: 13 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
# Changelog

## Unreleased
## Unreleased 2.2.0-RC3

### Feature

- Customer Detail Data Overlay
- implement new UI design for customer detail data overlay
- **Company Subscription Management**
- Extended the subscription detail page for customer to include more app-related information [#1051](https://github.com/eclipse-tractusx/portal-frontend/pull/1051)

### Change

Expand All @@ -15,9 +17,16 @@
- update the status(skipped) in application requests and necessary changes with checkbox in connector management [#1019](https://github.com/eclipse-tractusx/portal-frontend/pull/1019)
- **Technical User Management**
- Removed 'Inactive' filter in technical user management [#1046](https://github.com/eclipse-tractusx/portal-frontend/pull/1046)
- **Company Subscriptions**
- Update and bind API with filter options [#1062](https://github.com/eclipse-tractusx/portal-frontend/pull/1062)

### Bugfixes

- **Connector Management**
- fixed technical user selection
- **Technical User**
- Show appropriate error message.
- Fix closing page notification bar issue
- **IDP management**
- Fixed statusTag color in status coloumn [#978](https://github.com/eclipse-tractusx/portal-frontend/pull/978)
- **Connector Management**
Expand All @@ -30,6 +39,8 @@
- Fixed role upload does not work using Firefox [#1003](https://github.com/eclipse-tractusx/portal-frontend/pull/1003)
- **Technical User Management**
- Display technicalUserManagement button based on role validation [#1073](https://github.com/eclipse-tractusx/portal-frontend/pull/1073)
- **OSP Consent form**
- Display invited company name in OSP consent form (Previously hard coded with 'BMW') [#1083](https://github.com/eclipse-tractusx/portal-frontend/pull/1083)

## 2.2.0-RC2

Expand Down Expand Up @@ -86,6 +97,7 @@
- fixed unchanged text of button when user requests subscription [#985](https://github.com/eclipse-tractusx/portal-frontend/pull/985)
- fixed height for "Admin Service Detail" page content [#1001](https://github.com/eclipse-tractusx/portal-frontend/pull/1001)
- fixed onClick of "Overview My Apps" button in App Release Process screen [#1022](https://github.com/eclipse-tractusx/portal-frontend/issues/1022)
- Fix 400 Bad Request error in App Access Management -> Add Role search filter [#1057](https://github.com/eclipse-tractusx/portal-frontend/issues/1057)

## 2.1.0

Expand Down
39 changes: 35 additions & 4 deletions src/assets/locales/de/main.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"deactivate": "Deactivate",
"appReleaseProcess": "App Release Process",
"appSubscription": "App Subscription MGT",
"adminboard": "Admin Board",
"appAdminBoard": "Admin Board",
"appUserManagement": "App Benutzer Verwaltung",
"userdetails": "Benutzer Details",
"technicalUserManagement": "Technische Benutzer Verwaltung",
Expand Down Expand Up @@ -460,9 +460,38 @@
"action": "Aktion"
},
"filter": {
"requested": "angefordert",
"pending": "ausstehend",
"active": "aktiv",
"showAll": "zeige alles"
"inactive": "inaktiv",
"showAll": "Alles anzeigen"
}
},
"companySubscriptionsDetail": {
"language": "Verfügbare App Sprachen",
"description": "Description",
"subscribed": "Gezeichnet",
"requested": "Angefordert",
"declined": "Abgelehnt",
"howtouse": {
"heading": "Nutzungsbedingungen",
"message": "Die Nutzungsbedingungen regeln das Verhältnis zwischen App Interessenten und App Anbieter. Die Nutzungsbedingungen, sowie die Datenmodelle und weitere Informationen können durch Anklicken der Hyperlinks eingesehen werden."
},
"privacy": {
"heading": "Datenschutz",
"message": "Die Datenschutzerklärung gibt an, welche Daten die Geschäftsanwendungen erheben, handhaben und verarbeiten. Die nachstehenden Details zeigen explizit, welche Unternehmens-/Benutzerinformationen gesammelt/gespeichert werden.",
"notSupportedMessage": "Currently not yet supported",
"COMPANY_DATA": "Company Data",
"USER_DATA": "User Data",
"LOCATION": "Location",
"BROWSER_HISTORY": "Browser History",
"NONE": "None"
},
"table": {
"connector": "Verbinder",
"technicalUser": "Technischer Benutzer"
},
"technicalUser": {
"heading": "Technischer Details"
}
},
"appOverview": {
Expand Down Expand Up @@ -923,7 +952,9 @@
"userDetails": "Technischer Benutzer Details",
"description": "Beschreibung",
"spoc": "SPOC",
"permission": "Berechtigungen"
"permission": "Berechtigungen",
"companyServiceAccountTypeID": "Firmenservicekontotyp",
"authenticationType": "Authentifizierungstyp"
},
"status": {
"ACTIVE": "AKTIV",
Expand Down
2 changes: 1 addition & 1 deletion src/assets/locales/de/notification.json
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@
"servicemanagementboard": "Service Management Board öffnen",
"appOverview": "Get there",
"appSubscription": "Get there",
"adminboard": "Get there",
"appAdminBoard": "Get there",
"technicaluser": "Get there",
"companyRolesServiceProvider": "Get there",
"usermanagement": "Get there",
Expand Down
38 changes: 35 additions & 3 deletions src/assets/locales/en/main.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
"deactivate": "Deactivate",
"appReleaseProcess": "App Release Process",
"appSubscription": "App Subscription MGT",
"adminboard": "Admin Board",
"appAdminBoard": "Admin Board",
"appUserManagement": "App User Management",
"userdetails": "User Details",
"technicalUserManagement": "Technical User Management",
Expand Down Expand Up @@ -456,11 +456,41 @@
"action": "Action"
},
"filter": {
"requested": "requested",
"pending": "pending",
"active": "active",
"inactive": "inactive",
"showAll": "show all"
}
},
"companySubscriptionsDetail": {
"language": "Language",
"description": "Description",
"subscribed": "Subscribed",
"requested": "Requested",
"declined": "Declined",
"howtouse": {
"heading": "Documents",
"message": "The terms of use, as well as the data models and further information can be viewed by clicking on the documents shared below."
},
"privacy": {
"heading": "Privacy Policy",
"message": "The Privacy Policy states which data the business applications collects, handles and processes. The details below explicitly show which company / user information are getting collected/stored.",
"notSupportedMessage": "Currently not yet supported",
"COMPANY_DATA": "Company Data",
"COMPANY_DATA_ICON": "ApartmentIcon",
"USER_DATA": "User Data",
"LOCATION": "Location",
"BROWSER_HISTORY": "Browser History",
"NONE": "None"
},
"table": {
"connector": "Connector",
"technicalUser": "Technical User"
},
"technicalUser": {
"heading": "Technical Details"
}
},
"datamanagement": {
"message": "Contents of Data Management"
},
Expand Down Expand Up @@ -927,7 +957,9 @@
"userDetails": "Technical User Details",
"description": "Description",
"spoc": "SPOC",
"permission": "Permission"
"permission": "Permission",
"companyServiceAccountTypeID": "Company Service Account Type",
"authenticationType": "Authentication Type"
},
"status": {
"ACTIVE": "ACTIVE",
Expand Down
2 changes: 1 addition & 1 deletion src/assets/locales/en/notification.json
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@
"servicemanagementboard": "Get there",
"appOverview": "Get there",
"appSubscription": "Get there",
"adminboard": "Get there",
"appAdminBoard": "Get there",
"technicaluser": "Get there",
"companyRolesServiceProvider": "Get there",
"usermanagement": "Get there",
Expand Down
16 changes: 11 additions & 5 deletions src/components/overlays/DeleteTechnicalUser/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,13 +55,14 @@ export const DeleteTechnicalUser = ({ id }: { id: string }) => {
navigate(`/${PAGES.TECH_USER_MANAGEMENT}`)
}

const deleteUserError = (err: unknown) => {
// eslint-disable-next-line
const deleteUserError = (err: any) => {
const notification: PageNotificationsProps = {
open: true,
severity: SuccessErrorType.ERROR,
title:
'content.usermanagement.technicalUser.deleteTechUserNotificationErrorTitle',
description: err as string,
description: err.data.details[0].message,
}
dispatch(closeOverlay())
dispatch(setNotification(notification))
Expand All @@ -71,11 +72,16 @@ export const DeleteTechnicalUser = ({ id }: { id: string }) => {
const handleRemove = async () => {
if (!data) return
try {
await removeServiceAccount(data.serviceAccountId).unwrap()
deleteUserSuccess()
await removeServiceAccount(data.serviceAccountId)
.unwrap()
.then(() => {
deleteUserSuccess()
})
.catch((err) => {
deleteUserError(err)
})
} catch (err: unknown) {
deleteUserError(err)
console.log(err)
}
}
return data ? (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ export default function BoardContentDetails({ item }: { item: AppDetails }) {
color="secondary"
size="small"
onClick={() => {
navigate(`/${PAGES.ADMINBOARD}`)
navigate(`/${PAGES.APP_ADMIN_BOARD}`)
}}
>
{t('content.adminboardDetail.backToBoard')}
Expand Down
2 changes: 1 addition & 1 deletion src/components/pages/AdminBoardDetail/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default function AdminBoardDetail() {
backButtonLabel={t('global.actions.back')}
backButtonVariant="text"
onBackButtonClick={() => {
navigate(`/${PAGES.ADMINBOARD}`)
navigate(`/${PAGES.APP_ADMIN_BOARD}`)
}}
/>
</Box>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -152,11 +152,16 @@ export default function AppDetailHeader({ item }: AppDetailHeaderProps) {
? true
: false
}
onButtonClick={() =>
subscribeStatus === SubscriptionStatus.INACTIVE &&
user.roles.indexOf(ROLES.SUBSCRIBE_APP_MARKETPLACE) !== -1 &&
dispatch(show(OVERLAYS.APPMARKETPLACE_REQUEST, appId))
}
onButtonClick={() => {
if (buttonLabel === t('content.appdetail.requested')) {
return
}
return (
subscribeStatus === SubscriptionStatus.INACTIVE &&
user.roles.indexOf(ROLES.SUBSCRIBE_APP_MARKETPLACE) !== -1 &&
dispatch(show(OVERLAYS.APPMARKETPLACE_REQUEST, appId))
)
}}
/>
)
}
Expand Down
Loading

0 comments on commit a0001d4

Please sign in to comment.