From 99caba47f3411e8b3b3f21c8de2401383201254e Mon Sep 17 00:00:00 2001 From: Tobias Messner Date: Tue, 29 Oct 2024 19:16:16 +0100 Subject: [PATCH] feat: Add the ability to set a company logo and show environment badge Add the ability to set a logo that will be displayed next to the Capella text. Add a badge that can show the current environment with different UI depending on if the environment is productive. The badge is enabled during development by default. --- backend/capellacollab/configuration/models.py | 31 +++++- backend/capellacollab/configuration/routes.py | 2 +- backend/capellacollab/configuration/util.py | 27 +++++ backend/tests/test_navbar.py | 104 ++++++++++++++++++ docs/docs/admin/configure-for-your-org.md | 22 +++- frontend/.storybook/main.ts | 1 + .../.storybook/test-assets/narrow_logo.svg | 9 ++ frontend/.storybook/test-assets/wide_logo.svg | 9 ++ .../app/general/header/header.component.html | 12 +- .../app/general/header/header.component.ts | 2 + .../src/app/general/header/header.stories.ts | 7 ++ .../src/app/general/logo/logo.component.html | 24 ++++ .../src/app/general/logo/logo.component.ts | 17 +++ frontend/src/app/general/logo/logo.stories.ts | 87 +++++++++++++++ .../app/general/nav-bar/nav-bar.service.ts | 7 ++ .../src/app/openapi/.openapi-generator/FILES | 4 + frontend/src/app/openapi/model/badge-input.ts | 30 +++++ .../src/app/openapi/model/badge-output.ts | 30 +++++ .../src/app/openapi/model/badge-variant.ts | 21 ++++ frontend/src/app/openapi/model/models.ts | 4 + .../model/navbar-configuration-input.ts | 6 + .../model/navbar-configuration-output.ts | 6 + frontend/src/app/openapi/model/text.ts | 19 ++++ frontend/tailwind.config.cjs | 4 + 24 files changed, 473 insertions(+), 12 deletions(-) create mode 100644 backend/tests/test_navbar.py create mode 100644 frontend/.storybook/test-assets/narrow_logo.svg create mode 100644 frontend/.storybook/test-assets/wide_logo.svg create mode 100644 frontend/src/app/general/logo/logo.component.html create mode 100644 frontend/src/app/general/logo/logo.component.ts create mode 100644 frontend/src/app/general/logo/logo.stories.ts create mode 100644 frontend/src/app/openapi/model/badge-input.ts create mode 100644 frontend/src/app/openapi/model/badge-output.ts create mode 100644 frontend/src/app/openapi/model/badge-variant.ts create mode 100644 frontend/src/app/openapi/model/text.ts diff --git a/backend/capellacollab/configuration/models.py b/backend/capellacollab/configuration/models.py index 177d49307c..2f3bf4c1f7 100644 --- a/backend/capellacollab/configuration/models.py +++ b/backend/capellacollab/configuration/models.py @@ -12,7 +12,7 @@ from sqlalchemy import orm from capellacollab import core -from capellacollab.core import database +from capellacollab.core import DEVELOPMENT_MODE, database from capellacollab.core import pydantic as core_pydantic from capellacollab.users import models as users_models @@ -69,6 +69,27 @@ class CustomNavbarLink(NavbarLink): ) +class BadgeVariant(str, enum.Enum): + AUTO = "auto" + WARNING = "warning" + SUCCESS = "success" + + +class Badge(core_pydantic.BaseModelStrict): + show: bool = pydantic.Field( + default=True, + description="Show a badge with the current environment.", + ) + variant: BadgeVariant = pydantic.Field( + default=BadgeVariant.AUTO, + description="Color of the badge.", + ) + text: str | t.Literal["auto"] = pydantic.Field( + default="auto", + description="Text to display in the badge. Use 'auto' to display the environment name.", + ) + + class NavbarConfiguration(core_pydantic.BaseModelStrict): external_links: collections_abc.Sequence[ BuiltInNavbarLink | CustomNavbarLink @@ -105,6 +126,14 @@ class NavbarConfiguration(core_pydantic.BaseModelStrict): ), description="Links to display in the navigation bar.", ) + logo_url: str | None = pydantic.Field( + default=None, + description="URL to a logo to display in the navigation bar.", + ) + badge: Badge = pydantic.Field( + default=Badge(show=DEVELOPMENT_MODE), + description="Badge to display in the navigation bar.", + ) class FeedbackIntervalConfiguration(core_pydantic.BaseModelStrict): diff --git a/backend/capellacollab/configuration/routes.py b/backend/capellacollab/configuration/routes.py index bcda9ab5ba..afe858b2a2 100644 --- a/backend/capellacollab/configuration/routes.py +++ b/backend/capellacollab/configuration/routes.py @@ -35,7 +35,7 @@ def get_unified_config( metadata=util.get_metadata(cfg), feedback=util.get_feedback(cfg), beta=cfg.beta, - navbar=cfg.navbar, + navbar=util.get_navbar(cfg), ) diff --git a/backend/capellacollab/configuration/util.py b/backend/capellacollab/configuration/util.py index 13051e68b9..156d655141 100644 --- a/backend/capellacollab/configuration/util.py +++ b/backend/capellacollab/configuration/util.py @@ -2,6 +2,7 @@ # SPDX-License-Identifier: Apache-2.0 import capellacollab +from capellacollab import core from capellacollab.configuration.app import config from . import models @@ -33,3 +34,29 @@ def get_feedback( feedback.interval.enabled = False return feedback + + +def get_navbar( + global_config: models.GlobalConfiguration, +) -> models.NavbarConfiguration: + navbar_config = global_config.navbar + + if navbar_config.badge.show: + if navbar_config.badge.text == "auto": + if core.CLUSTER_DEVELOPMENT_MODE: + navbar_config.badge.text = "Cluster Development" + elif core.LOCAL_DEVELOPMENT_MODE: + navbar_config.badge.text = "Local Development" + else: + navbar_config.badge.text = ( + global_config.metadata.environment or "Unknown Environment" + ) + + if navbar_config.badge.variant == models.BadgeVariant.AUTO: + words = ["dev", "development", "unknown", "staging"] + if any(word in navbar_config.badge.text.lower() for word in words): + navbar_config.badge.variant = models.BadgeVariant.WARNING + else: + navbar_config.badge.variant = models.BadgeVariant.SUCCESS + + return navbar_config diff --git a/backend/tests/test_navbar.py b/backend/tests/test_navbar.py new file mode 100644 index 0000000000..7ab40f4a0a --- /dev/null +++ b/backend/tests/test_navbar.py @@ -0,0 +1,104 @@ +# SPDX-FileCopyrightText: Copyright DB InfraGO AG and contributors +# SPDX-License-Identifier: Apache-2.0 + +import pytest +from fastapi import testclient + +from capellacollab import core + + +@pytest.fixture(name="cluster_development_mode") +def fixture_cluster_development_mode(monkeypatch: pytest.MonkeyPatch): + monkeypatch.setattr(core, "CLUSTER_DEVELOPMENT_MODE", True) + monkeypatch.setattr(core, "DEVELOPMENT_MODE", True) + + +@pytest.fixture(name="local_development_mode") +def fixture_local_development_mode(monkeypatch: pytest.MonkeyPatch): + monkeypatch.setattr(core, "LOCAL_DEVELOPMENT_MODE", True) + monkeypatch.setattr(core, "DEVELOPMENT_MODE", True) + + +@pytest.mark.usefixtures("admin", "cluster_development_mode") +def test_cluster_dev_mode( + client: testclient.TestClient, +): + client.put( + "/api/v1/configurations/global", + json={ + "navbar": { + "badge": {"text": "auto", "variant": "auto", "show": True} + } + }, + ) + response = client.get("/api/v1/configurations/unified") + assert response.status_code == 200 + assert response.json()["navbar"]["badge"]["text"] == "Cluster Development" + assert response.json()["navbar"]["badge"]["variant"] == "warning" + + +@pytest.mark.usefixtures( + "admin", + "local_development_mode", +) +def test_local_dev_mode( + client: testclient.TestClient, +): + client.put( + "/api/v1/configurations/global", + json={ + "navbar": { + "badge": {"text": "auto", "variant": "auto", "show": True} + } + }, + ) + response = client.get("/api/v1/configurations/unified") + assert response.status_code == 200 + assert response.json()["navbar"]["badge"]["text"] == "Local Development" + assert response.json()["navbar"]["badge"]["variant"] == "warning" + + +@pytest.mark.usefixtures("admin") +def test_fallback_env_mode(client: testclient.TestClient): + response = client.put( + "/api/v1/configurations/global", + json={ + "metadata": { + "environment": "Fallback Environment", + }, + "navbar": { + "badge": {"text": "auto", "variant": "auto", "show": True} + }, + }, + ) + + assert response.status_code == 200 + + response = client.get("/api/v1/configurations/unified") + assert response.status_code == 200 + assert response.json()["navbar"]["badge"]["text"] == "Fallback Environment" + assert response.json()["navbar"]["badge"]["variant"] == "success" + + +@pytest.mark.usefixtures("admin") +def test_unknown_env_mode( + client: testclient.TestClient, +): + response = client.put( + "/api/v1/configurations/global", + json={ + "metadata": { + "environment": "", + }, + "navbar": { + "badge": {"text": "auto", "variant": "auto", "show": True} + }, + }, + ) + + assert response.status_code == 200 + + response = client.get("/api/v1/configurations/unified") + assert response.status_code == 200 + assert response.json()["navbar"]["badge"]["text"] == "Unknown Environment" + assert response.json()["navbar"]["badge"]["variant"] == "warning" diff --git a/docs/docs/admin/configure-for-your-org.md b/docs/docs/admin/configure-for-your-org.md index 0ae6cec0dc..78977fc4cb 100644 --- a/docs/docs/admin/configure-for-your-org.md +++ b/docs/docs/admin/configure-for-your-org.md @@ -27,11 +27,11 @@ metadata: environment: '-' ``` -## Navigation Bar +## Logo and Navigation Bar -You can edit the links in the navigation bar. This can be useful if you want to -link to external resources or if you are not using the default monitoring -setup. +You can edit the logo, badge, and links in the navigation bar. This can be +useful to brand the Collaboration Manager for your organization, remind users +which environment they are in, or link to external resources. ```yaml navbar: @@ -45,6 +45,11 @@ navbar: - name: Documentation service: documentation role: user + logo_url: null + badge: + show: true + variant: auto + text: auto ``` In addition to the default service links, you can add your own by using `href` @@ -63,6 +68,15 @@ hide the link from users without the appropriate role, it is not a security feature, and you should make sure that the linked service enforces the necessary access controls. +To show the logo in the navigation bar, set the `logo_url` field to the URL of +the image you want to use. + +The badge can be used to show the environment the user is in. The `variant` +field can be set to `auto` (it will be determined by the environment), +`success`, or `warning`. The `text` field will use the environment name if set +to `auto`, or you can specify a custom text. If you don't want to show the +badge, set `show` to `false`. + ## Feedback !!! info "Configure SMTP server for feedback" diff --git a/frontend/.storybook/main.ts b/frontend/.storybook/main.ts index 4dd5664be4..b32002f744 100644 --- a/frontend/.storybook/main.ts +++ b/frontend/.storybook/main.ts @@ -20,6 +20,7 @@ const config: StorybookConfig = { name: '@storybook/angular', options: {}, }, + staticDirs: [{ from: './test-assets', to: '/test-assets' }], core: { disableTelemetry: true, enableCrashReports: false, diff --git a/frontend/.storybook/test-assets/narrow_logo.svg b/frontend/.storybook/test-assets/narrow_logo.svg new file mode 100644 index 0000000000..2efe7e65f2 --- /dev/null +++ b/frontend/.storybook/test-assets/narrow_logo.svg @@ -0,0 +1,9 @@ + + + + + + diff --git a/frontend/.storybook/test-assets/wide_logo.svg b/frontend/.storybook/test-assets/wide_logo.svg new file mode 100644 index 0000000000..f338a4794e --- /dev/null +++ b/frontend/.storybook/test-assets/wide_logo.svg @@ -0,0 +1,9 @@ + + + + + + diff --git a/frontend/src/app/general/header/header.component.html b/frontend/src/app/general/header/header.component.html index eff6ada2e7..c94e00f938 100644 --- a/frontend/src/app/general/header/header.component.html +++ b/frontend/src/app/general/header/header.component.html @@ -16,12 +16,12 @@