diff --git a/CHANGELOG.md b/CHANGELOG.md index a82049ab..07714634 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,7 @@ please see [changelog_updates.md](docs/dev/changelog_updates.md). - Uptime Kuma is no longer mandatory and the status dashboard can be disabled - Added Owning Organization in Admin Connector Overview ([#355](https://github.com/sovity/authority-portal/issues/355)) - Added a button to change the password in the user settings ([PR #397](https://github.com/sovity/authority-portal/pull/397)) +- Removed link to frontend URL in all-connectors-view for authority- and operator admins ([PR #398](https://github.com/sovity/authority-portal/pull/398)) #### Patch diff --git a/authority-portal-backend/authority-portal-quarkus/src/main/kotlin/de/sovity/authorityportal/web/pages/connectormanagement/ConnectorManagementApiService.kt b/authority-portal-backend/authority-portal-quarkus/src/main/kotlin/de/sovity/authorityportal/web/pages/connectormanagement/ConnectorManagementApiService.kt index 50d92670..96fa3f67 100644 --- a/authority-portal-backend/authority-portal-quarkus/src/main/kotlin/de/sovity/authorityportal/web/pages/connectormanagement/ConnectorManagementApiService.kt +++ b/authority-portal-backend/authority-portal-quarkus/src/main/kotlin/de/sovity/authorityportal/web/pages/connectormanagement/ConnectorManagementApiService.kt @@ -107,10 +107,13 @@ class ConnectorManagementApiService( deploymentEnvironmentService.assertValidEnvId(environmentId) val connectors = connectorService.getConnectorsByEnvironment(environmentId) - return buildConnectorOverview(connectors) + return buildConnectorOverview(connectors, showFrontendUrl = false) } - private fun buildConnectorOverview(connectors: List): ConnectorOverviewResult { + private fun buildConnectorOverview( + connectors: List, + showFrontendUrl: Boolean = true + ): ConnectorOverviewResult { val orgNames = organizationService.getAllOrganizationNames() val connectorDtos = connectors.map { @@ -122,7 +125,7 @@ class ConnectorManagementApiService( environment = deploymentEnvironmentDtoService.findByIdOrThrow(it.environment), name = it.name, status = if (it.type == ConnectorType.CAAS) it.caasStatus.toDto() else it.onlineStatus.toDto(), - frontendUrl = it.frontendUrl + frontendUrl = if (showFrontendUrl) it.frontendUrl else null ) } return ConnectorOverviewResult(connectorDtos) diff --git a/authority-portal-backend/authority-portal-quarkus/src/test/kotlin/de/sovity/authorityportal/web/tests/services/connector/ConnectorManagementApiServiceTest.kt b/authority-portal-backend/authority-portal-quarkus/src/test/kotlin/de/sovity/authorityportal/web/tests/services/connector/ConnectorManagementApiServiceTest.kt index 05dac375..0f841979 100644 --- a/authority-portal-backend/authority-portal-quarkus/src/test/kotlin/de/sovity/authorityportal/web/tests/services/connector/ConnectorManagementApiServiceTest.kt +++ b/authority-portal-backend/authority-portal-quarkus/src/test/kotlin/de/sovity/authorityportal/web/tests/services/connector/ConnectorManagementApiServiceTest.kt @@ -158,6 +158,7 @@ class ConnectorManagementApiServiceTest { assertThat(result.connectors).hasSize(2) assertThat(result.connectors).noneMatch { it.environment.environmentId != "test" } assertThat(result.connectors).noneMatch { it.id == dummyDevConnectorId(0, 2) } + assertThat(result.connectors).allMatch { it.frontendUrl == null } assertThat(result.connectors[0].id).isEqualTo(dummyDevConnectorId(0, 0)) assertThat(result.connectors[1].id).isEqualTo(dummyDevConnectorId(0, 1)) @@ -191,6 +192,7 @@ class ConnectorManagementApiServiceTest { assertThat(result.connectors).noneMatch { it.environment.environmentId != "other-environment" } assertThat(result.connectors).noneMatch { it.id == dummyDevConnectorId(0, 0) } assertThat(result.connectors).noneMatch { it.id == dummyDevConnectorId(0, 1) } + assertThat(result.connectors).allMatch { it.frontendUrl == null } assertThat(result.connectors[0].id).isEqualTo(dummyDevConnectorId(0, 2)) assertThat(result.connectors[0].environment.environmentId).isEqualTo("other-environment") diff --git a/authority-portal-frontend/src/app/pages/authority-connector-list-page/authority-connector-list-page/authority-connector-list-page.component.html b/authority-portal-frontend/src/app/pages/authority-connector-list-page/authority-connector-list-page/authority-connector-list-page.component.html index ce318be2..8ac3f039 100644 --- a/authority-portal-frontend/src/app/pages/authority-connector-list-page/authority-connector-list-page/authority-connector-list-page.component.html +++ b/authority-portal-frontend/src/app/pages/authority-connector-list-page/authority-connector-list-page/authority-connector-list-page.component.html @@ -44,12 +44,6 @@ scope="col"> Environment - - - Frontend - @@ -95,19 +89,6 @@ {{ connector.environment.title }} - - - open_in_new - -