From 862fd234fb9a3fd1715db3486e2865918f130296 Mon Sep 17 00:00:00 2001
From: Frank Niessink <frank@niessink.com>
Date: Tue, 17 Oct 2023 11:03:17 +0200
Subject: [PATCH] Allow for showing all metrics on the reports overview page.
 Closes #7215.

---
 components/api_server/src/routes/report.py    | 36 +-------
 .../api_server/tests/routes/test_report.py    | 90 -------------------
 components/frontend/src/App.js                |  9 +-
 components/frontend/src/AppUI.js              | 30 +++++--
 components/frontend/src/PageContent.js        | 68 ++++++++------
 .../frontend/src/__fixtures__/fixtures.js     |  3 +-
 .../frontend/src/header_footer/Menubar.js     |  4 +-
 .../src/header_footer/SettingsPanel.js        |  5 +-
 .../src/header_footer/SettingsPanel.test.js   |  2 +-
 components/frontend/src/report/Report.js      |  7 +-
 .../frontend/src/report/ReportsOverview.js    | 85 +++++++++++++++---
 .../src/report/ReportsOverview.test.js        | 71 ++++++++++++---
 components/frontend/src/sharedPropTypes.js    | 18 ++--
 components/frontend/src/subject/Subject.js    |  9 +-
 .../frontend/src/subject/Subject.test.js      | 16 ++--
 .../frontend/src/subject/SubjectTable.js      | 29 ++++--
 .../frontend/src/subject/SubjectTitle.js      | 10 ++-
 components/frontend/src/subject/Subjects.js   | 10 ++-
 .../frontend/src/subject/SubjectsButtonRow.js |  5 +-
 components/frontend/src/utils.js              |  8 +-
 components/frontend/src/utils.test.js         | 22 ++---
 .../shared_code/src/shared/model/report.py    | 14 ++-
 .../shared_code/src/shared/model/subject.py   | 12 +--
 .../tests/shared/model/test_subject.py        | 24 -----
 docs/src/changelog.md                         |  1 +
 .../src/features/tagreport.feature            | 17 ----
 tests/feature_tests/src/steps/tagreport.py    | 25 ------
 27 files changed, 301 insertions(+), 329 deletions(-)
 delete mode 100644 tests/feature_tests/src/features/tagreport.feature
 delete mode 100644 tests/feature_tests/src/steps/tagreport.py

diff --git a/components/api_server/src/routes/report.py b/components/api_server/src/routes/report.py
index 5ea21fdb2b..52591feb32 100644
--- a/components/api_server/src/routes/report.py
+++ b/components/api_server/src/routes/report.py
@@ -11,7 +11,6 @@
 import requests
 from pymongo.database import Database
 
-from shared.utils.functions import iso_timestamp
 from shared.utils.type import ReportId
 from shared_data_model import DATA_MODEL
 from shared_data_model.parameters import PrivateToken
@@ -68,20 +67,12 @@ def get_report(database: Database, report_uuid: ReportId | None = None):
     data_model = latest_datamodel(database, date_time)
     reports = latest_reports_before_timestamp(database, data_model, date_time)
     summarized_reports = []
-
-    if report_uuid and report_uuid.startswith("tag-"):
-        report = tag_report(data_model, report_uuid[4:], reports)
-        if len(report.subjects) > 0:
+    for report in reports:
+        if not report_uuid or report["report_uuid"] == report_uuid:
             measurements = recent_measurements(database, report.metrics_dict, date_time)
             summarized_reports.append(report.summarize(measurements))
-    else:
-        for report in reports:
-            if not report_uuid or report["report_uuid"] == report_uuid:
-                measurements = recent_measurements(database, report.metrics_dict, date_time)
-                summarized_reports.append(report.summarize(measurements))
-            else:
-                summarized_reports.append(report)
-
+        else:
+            summarized_reports.append(report)
     hide_credentials(data_model, *summarized_reports)
     return {"ok": True, "reports": summarized_reports}
 
@@ -246,22 +237,3 @@ def get_report_issue_tracker_options(database: Database, report: Report):  # noq
     """Get options for the issue tracker attributes such as project key and issue type."""
     issue_tracker = report.issue_tracker()
     return issue_tracker.get_options().as_dict() | {"ok": True}
-
-
-def tag_report(data_model, tag: str, reports: list[Report]) -> Report:
-    """Create a report for a tag."""
-    subjects = {}
-    for report in reports:
-        for subject in report.subjects:
-            if tag_subject := subject.tag_subject(tag):
-                subjects[subject.uuid] = tag_subject
-
-    return Report(
-        data_model,
-        {
-            "title": f'Report for tag "{tag}"',
-            "report_uuid": f"tag-{tag}",
-            "timestamp": iso_timestamp(),
-            "subjects": subjects,
-        },
-    )
diff --git a/components/api_server/tests/routes/test_report.py b/components/api_server/tests/routes/test_report.py
index f99270e446..5db08d6ed3 100644
--- a/components/api_server/tests/routes/test_report.py
+++ b/components/api_server/tests/routes/test_report.py
@@ -1,7 +1,5 @@
 """Unit tests for the report routes."""
 
-from datetime import datetime, UTC
-from typing import cast
 from unittest.mock import Mock, patch
 import copy
 
@@ -438,84 +436,6 @@ def test_issue_status(self):
         report = get_report(self.database, REPORT_ID)["reports"][0]
         self.assertEqual(issue_status, report["subjects"][SUBJECT_ID]["metrics"][METRIC_ID]["issue_status"][0])
 
-    @patch("shared.utils.functions.datetime")
-    def test_get_tag_report(self, date_time):
-        """Test that a tag report can be retrieved."""
-        date_time.now.return_value = now = datetime.now(tz=UTC)
-        self.database.reports.find_one.return_value = {
-            "_id": "id",
-            "report_uuid": REPORT_ID,
-            "title": "Report",
-            "subjects": {
-                "subject_without_metrics": {"metrics": {}},
-                SUBJECT_ID: {
-                    "name": "Subject",
-                    "type": "software",
-                    "metrics": {
-                        "metric_with_tag": {"type": "violations", "tags": ["tag"]},
-                        "metric_without_tag": {"type": "violations", "tags": ["other tag"]},
-                    },
-                },
-            },
-        }
-        expected_counts = {"blue": 0, "red": 0, "green": 0, "yellow": 0, "grey": 0, "white": 1}
-        self.assertDictEqual(
-            {
-                "ok": True,
-                "reports": [
-                    {
-                        "summary": expected_counts,
-                        "title": 'Report for tag "tag"',
-                        "report_uuid": "tag-tag",
-                        "timestamp": now.replace(microsecond=0).isoformat(),
-                        "subjects": {
-                            SUBJECT_ID: {
-                                "name": "Report ❯ Subject",  # noqa: RUF001
-                                "type": "software",
-                                "metrics": {
-                                    "metric_with_tag": {
-                                        "status": None,
-                                        "status_start": None,
-                                        "scale": "count",
-                                        "sources": {},
-                                        "recent_measurements": [],
-                                        "latest_measurement": None,
-                                        "type": "violations",
-                                        "tags": ["tag"],
-                                    },
-                                },
-                            },
-                        },
-                    },
-                ],
-            },
-            get_report(self.database, "tag-tag"),
-        )
-
-    @patch("shared.utils.functions.datetime")
-    def test_no_empty_tag_report(self, date_time):
-        """Test that empty tag reports are omitted."""
-        date_time.now.return_value = datetime.now(tz=UTC)
-        self.database.reports.find.return_value = [
-            {
-                "_id": "id",
-                "report_uuid": REPORT_ID,
-                "title": "Report",
-                "subjects": {
-                    "subject_without_metrics": {"metrics": {}},
-                    SUBJECT_ID: {
-                        "name": "Subject",
-                        "type": "software",
-                        "metrics": {
-                            "metric_with_tag": {"type": "metric_type", "tags": ["tag"]},
-                            "metric_without_tag": {"type": "metric_type", "tags": ["other tag"]},
-                        },
-                    },
-                },
-            },
-        ]
-        self.assertDictEqual({"ok": True, "reports": []}, get_report(self.database, "tag-non-existing-tag"))
-
     def test_add_report(self):
         """Test that a report can be added."""
         self.assertTrue(post_report_new(self.database)["ok"])
@@ -564,16 +484,6 @@ def test_get_pdf_report(self, requests_get):
             timeout=120,
         )
 
-    @patch("requests.get")
-    def test_get_pdf_tag_report(self, requests_get):
-        """Test that a PDF version of a tag report can be retrieved."""
-        requests_get.return_value = Mock(content=b"PDF")
-        self.assertEqual(b"PDF", export_report_as_pdf(cast(ReportId, "tag-security")))
-        requests_get.assert_called_once_with(
-            "http://renderer:9000/api/render?path=tag-security%3Fhide_toasts%3Dtrue",
-            timeout=120,
-        )
-
     def test_delete_report(self):
         """Test that the report can be deleted."""
         self.assertEqual({"ok": True}, delete_report(self.database, REPORT_ID))
diff --git a/components/frontend/src/App.js b/components/frontend/src/App.js
index 44d1b268a5..b43a7e2598 100644
--- a/components/frontend/src/App.js
+++ b/components/frontend/src/App.js
@@ -6,7 +6,7 @@ import { get_report, get_reports_overview } from './api/report';
 import { nr_measurements_api } from './api/measurement';
 import { login } from './api/auth';
 import { showMessage, showConnectionMessage } from './widgets/toast';
-import { isValidDate_YYYYMMDD, registeredURLSearchParams, reportIsTagReport, toISODateStringInCurrentTZ } from './utils'
+import { isValidDate_YYYYMMDD, registeredURLSearchParams, toISODateStringInCurrentTZ } from './utils'
 import { AppUI } from './AppUI';
 import 'react-toastify/dist/ReactToastify.css';
 import './App.css';
@@ -132,13 +132,6 @@ class App extends Component {
     open_report(event, report_uuid) {
         event.preventDefault();
         this.history_push(encodeURI(report_uuid))
-        if (reportIsTagReport(report_uuid)) {
-            showMessage(
-                "info",
-                "Tag reports are read-only",
-                "You opened a report for a specific metric tag. These reports are generated dynamically. Editing is not possible."
-            )
-        }
         this.setState({ report_uuid: report_uuid, loading: true }, () => this.reload());
     }
 
diff --git a/components/frontend/src/AppUI.js b/components/frontend/src/AppUI.js
index 1d94ae55a3..b7c6fbca70 100644
--- a/components/frontend/src/AppUI.js
+++ b/components/frontend/src/AppUI.js
@@ -14,7 +14,8 @@ import { DataModel } from './context/DataModel';
 import { DarkMode } from './context/DarkMode';
 import { Permissions } from './context/Permissions';
 import { PageContent } from './PageContent';
-import { getReportsTags, getUserPermissions, reportIsTagReport, userPrefersDarkMode, useURLSearchQuery } from './utils'
+import { getReportsTags, getUserPermissions, userPrefersDarkMode, useURLSearchQuery } from './utils'
+import { datePropType, reportsPropType, stringsPropType } from './sharedPropTypes';
 
 export function AppUI({
     changed_fields,
@@ -47,10 +48,10 @@ export function AppUI({
         return () => mediaQueryList.removeEventListener("change", changeMode);
     }, [uiMode, setUIMode]);
 
-    const user_permissions = getUserPermissions(
-        user, email, reportIsTagReport(report_uuid), report_date, reports_overview.permissions || {}
-    )
-    const current_report = reports.filter((report) => report.report_uuid === report_uuid)[0] || null;
+    const user_permissions = getUserPermissions(user, email, report_date, reports_overview.permissions || {})
+    const atReportsOverview = report_uuid === ""
+    const current_report = atReportsOverview ? null : reports.filter((report) => report.report_uuid === report_uuid)[0];
+    const metricsToHideDefault = atReportsOverview ? "all" : "none"
     // Make the settings changeable per report (and separately for the reports overview) by adding the report UUID as
     // postfix to the settings key:
     const urlSearchQueryKeyPostfix = report_uuid ? `_${report_uuid}` : ""
@@ -58,7 +59,7 @@ export function AppUI({
     const [dateOrder, setDateOrder] = useURLSearchQuery("date_order" + urlSearchQueryKeyPostfix, "string", "descending");
     const [hiddenColumns, toggleHiddenColumn, clearHiddenColumns] = useURLSearchQuery("hidden_columns" + urlSearchQueryKeyPostfix, "array");
     const [hiddenTags, toggleHiddenTag, clearHiddenTags] = useURLSearchQuery("hidden_tags" + urlSearchQueryKeyPostfix, "array");
-    const [metricsToHide, setMetricsToHide] = useURLSearchQuery("metrics_to_hide" + urlSearchQueryKeyPostfix, "string", "none");
+    const [metricsToHide, setMetricsToHide] = useURLSearchQuery("metrics_to_hide" + urlSearchQueryKeyPostfix, "string", metricsToHideDefault);
     const [nrDates, setNrDates] = useURLSearchQuery("nr_dates" + urlSearchQueryKeyPostfix, "integer", 1);
     const [sortColumn, setSortColumn] = useURLSearchQuery("sort_column" + urlSearchQueryKeyPostfix, "string", null);
     const [sortDirection, setSortDirection] = useURLSearchQuery("sort_direction" + urlSearchQueryKeyPostfix, "string", "ascending");
@@ -95,7 +96,6 @@ export function AppUI({
 
     const darkMode = userPrefersDarkMode(uiMode);
     const backgroundColor = darkMode ? "rgb(40, 40, 40)" : "white"
-    const atReportsOverview = report_uuid === ""
     return (
         <div style={{ display: "flex", minHeight: "100vh", flexDirection: "column", backgroundColor: backgroundColor }}>
             <DarkMode.Provider value={darkMode}>
@@ -182,5 +182,19 @@ export function AppUI({
     )
 }
 AppUI.propTypes = {
-    openReportsOverview: PropTypes.func
+    changed_fields: stringsPropType,
+    datamodel: PropTypes.object,
+    email: PropTypes.string,
+    handleDateChange: PropTypes.func,
+    last_update: datePropType,
+    loading: PropTypes.bool,
+    open_report: PropTypes.func,
+    openReportsOverview: PropTypes.func,
+    reload: PropTypes.func,
+    report_date: datePropType,
+    report_uuid: PropTypes.string,
+    reports: reportsPropType,
+    reports_overview: PropTypes.object,
+    set_user: PropTypes.func,
+    user: PropTypes.string
 }
\ No newline at end of file
diff --git a/components/frontend/src/PageContent.js b/components/frontend/src/PageContent.js
index 83b111013d..7179ddca6f 100644
--- a/components/frontend/src/PageContent.js
+++ b/components/frontend/src/PageContent.js
@@ -5,7 +5,7 @@ import { Segment } from './semantic_ui_react_wrappers';
 import { Report } from './report/Report';
 import { ReportsOverview } from './report/ReportsOverview';
 import { get_measurements } from './api/measurement';
-import { metricsToHidePropType, stringsPropType } from './sharedPropTypes';
+import { datePropType, issueSettingsPropType, metricsToHidePropType, reportsPropType, sortDirectionPropType, stringsPropType } from './sharedPropTypes';
 
 function getColumnDates(reportDate, dateInterval, dateOrder, nrDates) {
     const baseDate = reportDate ? new Date(reportDate) : new Date();
@@ -29,8 +29,8 @@ export function PageContent({
     handleSort,
     hiddenColumns,
     hiddenTags,
-    metricsToHide,
     issueSettings,
+    metricsToHide,
     loading,
     nrDates,
     nrMeasurements,
@@ -60,45 +60,63 @@ export function PageContent({
     if (loading) {
         content = <Segment basic placeholder aria-label="Loading..."><Loader active size="massive" /></Segment>
     } else {
+        const commonProps = {
+            changed_fields: changed_fields,
+            dates: dates,
+            handleSort: handleSort,
+            hiddenColumns: hiddenColumns,
+            hiddenTags: hiddenTags,
+            issueSettings: issueSettings,
+            measurements: measurements,
+            metricsToHide: metricsToHide,
+            reload: reload,
+            reports: reports,
+            report_date: report_date,
+            sortColumn: sortColumn,
+            sortDirection: sortDirection,
+            toggleHiddenTag: toggleHiddenTag,
+            toggleVisibleDetailsTab: toggleVisibleDetailsTab,
+            visibleDetailsTabs: visibleDetailsTabs
+        }
         if (report_uuid) {
             content = <Report
-                changed_fields={changed_fields}
-                dates={dates}
                 openReportsOverview={openReportsOverview}
-                handleSort={handleSort}
-                hiddenColumns={hiddenColumns}
-                hiddenTags={hiddenTags}
-                metricsToHide={metricsToHide}
-                issueSettings={issueSettings}
-                measurements={measurements}
-                reload={reload}
                 report={current_report}
-                reports={reports}
-                report_date={report_date}
-                sortColumn={sortColumn}
-                sortDirection={sortDirection}
-                toggleHiddenTag={toggleHiddenTag}
-                toggleVisibleDetailsTab={toggleVisibleDetailsTab}
-                visibleDetailsTabs={visibleDetailsTabs}
+                {...commonProps}
             />
         } else {
             content = <ReportsOverview
-                dates={dates}
-                hiddenTags={hiddenTags}
-                measurements={measurements}
                 open_report={open_report}
-                reload={reload}
-                reports={reports}
                 reports_overview={reports_overview}
-                report_date={report_date}
+                {...commonProps}
             />
         }
     }
     return <Container fluid className="MainContainer">{content}</Container>
 }
 PageContent.propTypes = {
+    changed_fields: stringsPropType,
+    current_report: PropTypes.object,
+    dateInterval: PropTypes.number,
+    dateOrder: sortDirectionPropType,
+    handleSort: PropTypes.func,
+    hiddenColumns: stringsPropType,
     hiddenTags: stringsPropType,
+    issueSettings: issueSettingsPropType,
     metricsToHide: metricsToHidePropType,
+    loading: PropTypes.bool,
+    nrDates: PropTypes.number,
+    nrMeasurements: PropTypes.number,
+    open_report: PropTypes.func,
     openReportsOverview: PropTypes.func,
-    toggleHiddenTag: PropTypes.func
+    reload: PropTypes.func,
+    report_date: datePropType,
+    report_uuid: PropTypes.string,
+    reports: reportsPropType,
+    reports_overview: PropTypes.object,
+    sortColumn: PropTypes.string,
+    sortDirection: sortDirectionPropType,
+    toggleHiddenTag: PropTypes.func,
+    toggleVisibleDetailsTab: PropTypes.func,
+    visibleDetailsTabs: stringsPropType
 }
diff --git a/components/frontend/src/__fixtures__/fixtures.js b/components/frontend/src/__fixtures__/fixtures.js
index 25b1f6b78c..dab539a18f 100644
--- a/components/frontend/src/__fixtures__/fixtures.js
+++ b/components/frontend/src/__fixtures__/fixtures.js
@@ -44,5 +44,6 @@ export const report = {
                 }
             }
         }
-    }
+    },
+    title: "Report title"
 }
diff --git a/components/frontend/src/header_footer/Menubar.js b/components/frontend/src/header_footer/Menubar.js
index b2cfe8b406..013f37be6d 100644
--- a/components/frontend/src/header_footer/Menubar.js
+++ b/components/frontend/src/header_footer/Menubar.js
@@ -6,7 +6,7 @@ import FocusLock from 'react-focus-lock';
 import { login, logout } from '../api/auth';
 import { Avatar } from '../widgets/Avatar';
 import { DatePicker } from '../widgets/DatePicker';
-import { datePropType, uiModePropType } from '../sharedPropTypes';
+import { datePropType, stringsPropType, uiModePropType } from '../sharedPropTypes';
 import { UIModeMenu } from './UIModeMenu';
 import './Menubar.css';
 
@@ -178,5 +178,5 @@ Menubar.propTypes = {
     setUIMode: PropTypes.func,
     uiMode: uiModePropType,
     user: PropTypes.string,
-    visibleDetailsTabs: PropTypes.array
+    visibleDetailsTabs: stringsPropType
 }
diff --git a/components/frontend/src/header_footer/SettingsPanel.js b/components/frontend/src/header_footer/SettingsPanel.js
index 9d0257e62a..a7c1de8e36 100644
--- a/components/frontend/src/header_footer/SettingsPanel.js
+++ b/components/frontend/src/header_footer/SettingsPanel.js
@@ -494,6 +494,7 @@ function ResetSettingsButton(
         visibleDetailsTabs
     }
 ) {
+    const metricsToHideDefault = atReportsOverview ? "all" : "none"
     return (
         <Button
             disabled={
@@ -507,7 +508,7 @@ function ResetSettingsButton(
                 !issueSettings.showIssueDueDate &&
                 !issueSettings.showIssueRelease &&
                 !issueSettings.showIssueSprint &&
-                metricsToHide === "none" &&
+                metricsToHide === metricsToHideDefault &&
                 nrDates === 1 &&
                 reportDate === null &&
                 sortColumn === null &&
@@ -523,7 +524,7 @@ function ResetSettingsButton(
                 setNrDates(1);
                 setDateInterval(7);
                 setDateOrder("descending");
-                setMetricsToHide("none");
+                setMetricsToHide(metricsToHideDefault);
                 setShowIssueCreationDate(false);
                 setShowIssueSummary(false);
                 setShowIssueUpdateDate(false);
diff --git a/components/frontend/src/header_footer/SettingsPanel.test.js b/components/frontend/src/header_footer/SettingsPanel.test.js
index 6824de1c86..5480df74d8 100644
--- a/components/frontend/src/header_footer/SettingsPanel.test.js
+++ b/components/frontend/src/header_footer/SettingsPanel.test.js
@@ -59,7 +59,7 @@ it('resets the settings', () => {
     expect(props.setDateInterval).toHaveBeenCalledWith(7)
     expect(props.setDateOrder).toHaveBeenCalledWith("descending")
     expect(props.setNrDates).toHaveBeenCalledWith(1)
-    expect(props.setMetricsToHide).toHaveBeenCalledWith("none")
+    expect(props.setMetricsToHide).toHaveBeenCalledWith("all")
     expect(props.setShowIssueCreationDate).toHaveBeenCalledWith(false)
     expect(props.setShowIssueSummary).toHaveBeenCalledWith(false)
     expect(props.setShowIssueUpdateDate).toHaveBeenCalledWith(false)
diff --git a/components/frontend/src/report/Report.js b/components/frontend/src/report/Report.js
index bddca6afd4..f1a74a788e 100644
--- a/components/frontend/src/report/Report.js
+++ b/components/frontend/src/report/Report.js
@@ -5,6 +5,8 @@ import {
     datesPropType,
     issueSettingsPropType,
     metricsToHidePropType,
+    reportPropType,
+    reportsPropType,
     sortDirectionPropType,
     stringsPropType
 } from '../sharedPropTypes';
@@ -169,6 +171,7 @@ export function Report({
                 reload={reload}
             />
             <Subjects
+                atReportsOverview={false}
                 changed_fields={changed_fields}
                 dates={dates}
                 handleSort={handleSort}
@@ -201,9 +204,9 @@ Report.propTypes = {
     metricsToHide: metricsToHidePropType,
     openReportsOverview: PropTypes.func,
     reload: PropTypes.func,
-    report: PropTypes.object,
+    report: reportPropType,
     report_date: datePropType,
-    reports: PropTypes.array,
+    reports: reportsPropType,
     sortColumn: PropTypes.string,
     sortDirection: sortDirectionPropType,
     toggleHiddenTag: PropTypes.func,
diff --git a/components/frontend/src/report/ReportsOverview.js b/components/frontend/src/report/ReportsOverview.js
index e5ed2d4305..8b20fd1107 100644
--- a/components/frontend/src/report/ReportsOverview.js
+++ b/components/frontend/src/report/ReportsOverview.js
@@ -5,6 +5,7 @@ import { EDIT_REPORT_PERMISSION, ReadOnlyOrEditable } from '../context/Permissio
 import { CardDashboard } from '../dashboard/CardDashboard';
 import { LegendCard } from '../dashboard/LegendCard';
 import { MetricSummaryCard } from '../dashboard/MetricSummaryCard';
+import { Subjects } from '../subject/Subjects';
 import { CommentSegment } from '../widgets/CommentSegment';
 import { Tag } from '../widgets/Tag';
 import { add_report, set_reports_attribute, copy_report } from '../api/report';
@@ -12,7 +13,15 @@ import { ReportsOverviewTitle } from './ReportsOverviewTitle';
 import { AddButton, CopyButton } from '../widgets/Button';
 import { report_options } from '../widgets/menu_options';
 import { getMetricTags, getReportsTags, nrMetricsInReport, STATUS_COLORS, sum } from '../utils';
-import { datePropType, datesPropType, stringsPropType } from '../sharedPropTypes';
+import {
+    datePropType,
+    datesPropType,
+    issueSettingsPropType,
+    metricsToHidePropType,
+    reportsPropType,
+    sortDirectionPropType,
+    stringsPropType
+} from '../sharedPropTypes';
 import { ReportsOverviewErrorMessage } from './ReportErrorMessage';
 import { metricStatusOnDate } from './report_utils';
 
@@ -43,7 +52,7 @@ function summarizeReportsOnDate(reports, measurements, date, tag) {
 }
 
 
-function ReportsDashboard({ dates, hiddenTags, reports, open_report, measurements, layout, reload }) {
+function ReportsDashboard({ dates, hiddenTags, reports, onClickTag, open_report, measurements, layout, reload }) {
     let nrMetrics = 0
     const reportSummary = {}
     reports.forEach((report) => {
@@ -76,7 +85,7 @@ function ReportsDashboard({ dates, hiddenTags, reports, open_report, measurement
             key={tag}
             header={<Tag tag={tag} />}
             maxY={nrMetrics}
-            onClick={(e) => open_report(e, `tag-${tag}`)}
+            onClick={() => onClickTag(tag)}
             summary={tagSummary[tag]}
         />
     );
@@ -91,14 +100,36 @@ function ReportsDashboard({ dates, hiddenTags, reports, open_report, measurement
 ReportsDashboard.propTypes = {
     dates: datesPropType,
     hiddenTags: stringsPropType,
-    reports: PropTypes.array,
-    open_report: PropTypes.func,
-    measurements: PropTypes.array,
     layout: PropTypes.array,
-    reload: PropTypes.func
+    measurements: PropTypes.array,
+    onClickTag: PropTypes.func,
+    open_report: PropTypes.func,
+    reload: PropTypes.func,
+    reports: reportsPropType
 }
 
-export function ReportsOverview({ dates, hiddenTags, measurements, reports, open_report, report_date, reports_overview, reload }) {
+export function ReportsOverview(
+    {
+        changed_fields,
+        dates,
+        handleSort,
+        hiddenColumns,
+        hiddenTags,
+        issueSettings,
+        measurements,
+        metricsToHide,
+        open_report,
+        reload,
+        reports,
+        report_date,
+        reports_overview,
+        sortColumn,
+        sortDirection,
+        toggleHiddenTag,
+        toggleVisibleDetailsTab,
+        visibleDetailsTabs
+    }
+) {
     if (reports.length === 0 && report_date !== null) {
         return <ReportsOverviewErrorMessage reportDate={report_date} />
     }
@@ -113,11 +144,33 @@ export function ReportsOverview({ dates, hiddenTags, measurements, reports, open
                 hiddenTags={hiddenTags}
                 layout={reports_overview.layout ?? []}
                 measurements={reversedMeasurements}
+                onClickTag={(tag) => {
+                    const tagsToToggle = hiddenTags?.length > 0 ? hiddenTags : getReportsTags(reports)
+                    toggleHiddenTag(...tagsToToggle.filter((visibleTag) => visibleTag !== tag))
+                }}
                 open_report={open_report}
                 reload={reload}
                 reports={reports}
             />
-
+            <Subjects
+                atReportsOverview={true}
+                changed_fields={changed_fields}
+                dates={dates}
+                handleSort={handleSort}
+                hiddenColumns={hiddenColumns}
+                hiddenTags={hiddenTags}
+                issueSettings={issueSettings}
+                measurements={measurements}
+                metricsToHide={metricsToHide}
+                reload={reload}
+                reportsToShow={reports}
+                report_date={report_date}
+                reports={reports}
+                sortColumn={sortColumn}
+                sortDirection={sortDirection}
+                toggleVisibleDetailsTab={toggleVisibleDetailsTab}
+                visibleDetailsTabs={visibleDetailsTabs}
+            />
             <ReadOnlyOrEditable requiredPermissions={[EDIT_REPORT_PERMISSION]} editableComponent={
                 <Segment basic>
                     <AddButton
@@ -136,12 +189,22 @@ export function ReportsOverview({ dates, hiddenTags, measurements, reports, open
     )
 }
 ReportsOverview.propTypes = {
+    changed_fields: stringsPropType,
     dates: datesPropType,
+    handleSort: PropTypes.func,
+    hiddenColumns: stringsPropType,
     hiddenTags: stringsPropType,
+    issueSettings: issueSettingsPropType,
     measurements: PropTypes.array,
-    reports: PropTypes.array,
+    metricsToHide: metricsToHidePropType,
+    reports: reportsPropType,
     open_report: PropTypes.func,
+    reload: PropTypes.func,
     report_date: datePropType,
     reports_overview: PropTypes.object,
-    reload: PropTypes.func
+    sortColumn: PropTypes.string,
+    sortDirection: sortDirectionPropType,
+    toggleHiddenTag: PropTypes.func,
+    toggleVisibleDetailsTab: PropTypes.func,
+    visibleDetailsTabs: stringsPropType
 }
diff --git a/components/frontend/src/report/ReportsOverview.test.js b/components/frontend/src/report/ReportsOverview.test.js
index 961a416960..0264582c17 100644
--- a/components/frontend/src/report/ReportsOverview.test.js
+++ b/components/frontend/src/report/ReportsOverview.test.js
@@ -1,9 +1,10 @@
 import React from 'react';
 import { act, fireEvent, render, screen } from '@testing-library/react';
-import { ReportsOverview } from './ReportsOverview';
+import { DataModel } from '../context/DataModel';
 import { EDIT_REPORT_PERMISSION, Permissions } from '../context/Permissions';
 import * as fetch_server_api from '../api/fetch_server_api';
 import { mockGetAnimations } from '../dashboard/MockAnimations';
+import { ReportsOverview } from './ReportsOverview';
 
 beforeEach(() => {
     fetch_server_api.fetch_server_api = jest.fn().mockReturnValue({ then: jest.fn().mockReturnValue({ finally: jest.fn() }) });
@@ -12,16 +13,30 @@ beforeEach(() => {
 
 afterEach(() => jest.restoreAllMocks());
 
-function render_reports_overview(reports, reportsOverview, reportDate) {
+const datamodel = {
+    subjects: {
+        subject_type: { name: "Subject type", metrics: [ "metric_type"] }
+    },
+    metrics: {
+        metric_type: { name: "Metric type", tags: [] }
+    }
+}
+
+function render_reports_overview(reports, reportsOverview, reportDate, toggleHiddenTag) {
     render(
         <Permissions.Provider value={[EDIT_REPORT_PERMISSION]}>
-            <ReportsOverview
-                dates={[reportDate || new Date()]}
-                measurements={[{ status: "target_met" }]}
-                report_date={reportDate || null}
-                reports={reports}
-                reports_overview={reportsOverview}
-            />
+            <DataModel.Provider value={datamodel}>
+                <ReportsOverview
+                    dates={[reportDate || new Date()]}
+                    hiddenColumns={[]}
+                    measurements={[{ status: "target_met" }]}
+                    report_date={reportDate || null}
+                    reports={reports}
+                    reports_overview={reportsOverview}
+                    toggleHiddenTag={toggleHiddenTag ?? jest.fn()}
+                    visibleDetailsTabs={[]}
+                />
+            </DataModel.Provider>
         </Permissions.Provider>
     )
 }
@@ -45,13 +60,45 @@ it('shows the comment', async () => {
     expect(screen.getAllByText(/Commentary/).length).toBe(1);
 });
 
+const reports = [
+    {
+        report_uuid: "report_uuid",
+        subjects: {
+            subject_uuid: {
+                metrics: {
+                    metric_uuid: {
+                        recent_measurements: [],
+                        tags: ["Foo"],
+                        type: "metric_type"
+                    },
+                    metric_uuid2: {
+                        recent_measurements: [],
+                        tags: ["Bar"],
+                        type: "metric_type"
+                    }
+                },
+                type: "subject_type",
+            }
+        }
+    }
+]
+
+const reportsOverview = { title: "Overview", permissions: {} }
+
 it('shows the report tag cards', async () => {
-    const reports = [{ subjects: { subject_uuid: { metrics: { metric_uuid: { tags: ["Tag"] } } } } }]
-    const reportsOverview = { title: "Overview", permissions: {} }
     await act(async () => render_reports_overview(reports, reportsOverview))
-    expect(screen.getAllByText(/Tag/).length).toBe(1);
+    expect(screen.getAllByText(/Foo/).length).toBe(2);  // One in the dashboard, one in the table of metrics
 });
 
+it('hides the report tag cards', async () => {
+    const toggleHiddenTag = jest.fn()
+    await act(async () => render_reports_overview(reports, reportsOverview, new Date(), toggleHiddenTag))
+    expect(screen.getAllByText(/Foo/).length).toBe(2)  // One in the dashboard, one in the table of metrics
+    expect(screen.getAllByText(/Bar/).length).toBe(2)  // One in the dashboard, one in the table of metrics
+    fireEvent.click(screen.getAllByText(/Foo/)[0])
+    expect(toggleHiddenTag).toHaveBeenLastCalledWith("Bar")
+})
+
 it('adds a report', async () => {
     fetch_server_api.fetch_server_api = jest.fn().mockResolvedValue({ ok: true });
     await act(async () => render_reports_overview([], {}));
diff --git a/components/frontend/src/sharedPropTypes.js b/components/frontend/src/sharedPropTypes.js
index 22d4bd4efd..4b9298f891 100644
--- a/components/frontend/src/sharedPropTypes.js
+++ b/components/frontend/src/sharedPropTypes.js
@@ -1,5 +1,9 @@
 import PropTypes from 'prop-types';
 
+export const datePropType = PropTypes.instanceOf(Date)
+
+export const datesPropType = PropTypes.arrayOf(datePropType)
+
 export const issueSettingsPropType = PropTypes.shape({
     showIssueCreationDate: PropTypes.bool,
     showIssueDueDate: PropTypes.bool,
@@ -9,14 +13,16 @@ export const issueSettingsPropType = PropTypes.shape({
     showIssueUpdateDate: PropTypes.bool
 })
 
-export const sortDirectionPropType = PropTypes.oneOf(["ascending", "descending"])
-
-export const datePropType = PropTypes.instanceOf(Date)
+export const metricsToHidePropType = PropTypes.oneOf(["none", "all", "no_action_needed"])
 
-export const datesPropType = PropTypes.arrayOf(datePropType)
+export const reportPropType = PropTypes.shape({
+    issue_tracker: PropTypes.object,
+})
 
-export const uiModePropType = PropTypes.oneOf(["dark", "light", "follow_os"])
+export const reportsPropType = PropTypes.arrayOf(reportPropType)
 
-export const metricsToHidePropType = PropTypes.oneOf(["none", "all", "no_action_needed"])
+export const sortDirectionPropType = PropTypes.oneOf(["ascending", "descending"])
 
 export const stringsPropType = PropTypes.arrayOf(PropTypes.string)
+
+export const uiModePropType = PropTypes.oneOf(["dark", "light", "follow_os"])
diff --git a/components/frontend/src/subject/Subject.js b/components/frontend/src/subject/Subject.js
index 008939731e..c501f8c520 100644
--- a/components/frontend/src/subject/Subject.js
+++ b/components/frontend/src/subject/Subject.js
@@ -6,6 +6,8 @@ import {
     datesPropType,
     issueSettingsPropType,
     metricsToHidePropType,
+    reportPropType,
+    reportsPropType,
     sortDirectionPropType,
     stringsPropType
 } from '../sharedPropTypes';
@@ -96,6 +98,7 @@ function sortMetrics(datamodel, metrics, sortDirection, sortColumn, report, meas
 }
 
 export function Subject({
+    atReportsOverview,
     changed_fields,
     dates,
     firstSubject,
@@ -129,6 +132,7 @@ export function Subject({
         <div id={subject_uuid}>
             <div className="sticky">
                 <SubjectTitle
+                    atReportsOverview={atReportsOverview}
                     report={report}
                     subject={subject}
                     subject_uuid={subject_uuid}
@@ -161,6 +165,7 @@ export function Subject({
     )
 }
 Subject.propTypes = {
+    atReportsOverview: PropTypes.bool,
     changed_fields: stringsPropType,
     dates: datesPropType,
     firstSubject: PropTypes.bool,
@@ -171,9 +176,9 @@ Subject.propTypes = {
     lastSubject: PropTypes.bool,
     measurements: PropTypes.array,
     metricsToHide: metricsToHidePropType,
-    report: PropTypes.object,
+    report: reportPropType,
     report_date: datePropType,
-    reports: PropTypes.array,
+    reports: reportsPropType,
     sortColumn: PropTypes.string,
     sortDirection: sortDirectionPropType,
     subject_uuid: PropTypes.string,
diff --git a/components/frontend/src/subject/Subject.test.js b/components/frontend/src/subject/Subject.test.js
index 64cb661626..dafe5056e0 100644
--- a/components/frontend/src/subject/Subject.test.js
+++ b/components/frontend/src/subject/Subject.test.js
@@ -3,10 +3,11 @@ import { Subject } from "./Subject";
 import { DataModel } from "../context/DataModel";
 import { datamodel, report } from "../__fixtures__/fixtures";
 
-function renderSubject(dates, hiddenTags, metricsToHide, sortColumn, sortDirection, reportDate) {
+function renderSubject(atReportsOverview, dates, hiddenTags, metricsToHide, sortColumn, sortDirection, reportDate) {
     render(
         <DataModel.Provider value={datamodel}>
             <Subject
+                atReportsOverview={atReportsOverview}
                 dates={dates}
                 handleSort={() => { /* Dummy implementation */ }}
                 hiddenTags={hiddenTags ?? []}
@@ -25,17 +26,22 @@ function renderSubject(dates, hiddenTags, metricsToHide, sortColumn, sortDirecti
 }
 
 it('shows the subject title', async () => {
-    await act(async () => { renderSubject([new Date(2022, 3, 26)]) });
+    await act(async () => { renderSubject(false, [new Date(2022, 3, 26)]) });
     expect(screen.queryAllByText("Subject 1 title").length).toBe(1);
 })
 
+it('shows the subject title at the reports overview', async () => {
+    await act(async () => { renderSubject(true, [new Date(2022, 3, 26)]) });
+    expect(screen.queryAllByText("Report title ❯ Subject 1 title").length).toBe(1);
+})
+
 it('hides metrics not requiring action', async () => {
-    await act(async () => { renderSubject([new Date(2022, 3, 26)], [], "no_action_needed") });
+    await act(async () => { renderSubject(false, [new Date(2022, 3, 26)], [], "no_action_needed") });
     expect(screen.queryAllByText(/M\d/).length).toBe(1);
 })
 
 it('hides the subject if all metrics are hidden', async () => {
-    await act(async () => { renderSubject([], ["tag", "other tag"]) });
+    await act(async () => { renderSubject(false, [], ["tag", "other tag"]) });
     expect(screen.queryAllByText("Subject 1 title").length).toBe(0);
 })
 
@@ -46,7 +52,7 @@ function expectOrder(metricNames) {
 for (const attribute of ["name", "measurement", "target", "comment", "source", "issues", "tags", "unit", "status", "time_left", "overrun"]) {
     for (const order of ["ascending", "descending"]) {
         it('sorts metrics by attribute', async () => {
-            await act(async () => { renderSubject([], [], "none", attribute, order) });
+            await act(async () => { renderSubject(false, [], [], "none", attribute, order) });
             expectOrder(order === "ascending" ? ["M1", "M2"] : ["M2", "M1"])
         })
     }
diff --git a/components/frontend/src/subject/SubjectTable.js b/components/frontend/src/subject/SubjectTable.js
index 9a3157aa2b..4e8ff920fb 100644
--- a/components/frontend/src/subject/SubjectTable.js
+++ b/components/frontend/src/subject/SubjectTable.js
@@ -1,4 +1,5 @@
 import React, { useContext } from 'react';
+import PropTypes from 'prop-types';
 import { Table } from '../semantic_ui_react_wrappers';
 import { DataModel } from "../context/DataModel";
 import { DarkMode } from "../context/DarkMode";
@@ -17,6 +18,7 @@ import { formatMetricScale, get_metric_name, getMetricTags, getMetricUnit } from
 import { SubjectTableFooter } from './SubjectTableFooter';
 import { SubjectTableHeader } from './SubjectTableHeader';
 import "./SubjectTable.css"
+import { datePropType, datesPropType, issueSettingsPropType, reportsPropType, sortDirectionPropType, stringsPropType } from '../sharedPropTypes';
 
 function MeasurementCells({ dates, metric, metric_uuid, measurements }) {
     return (
@@ -123,11 +125,7 @@ export function SubjectTable({
                             {nrDates > 1 && !hiddenColumns.includes("overrun") && <Table.Cell style={style}><Overrun metric={metric} metric_uuid={metric_uuid} report={report} measurements={measurements} dates={dates} /></Table.Cell>}
                             {!hiddenColumns.includes("comment") && <Table.Cell style={style}><div style={{wordBreak: "break-word"}} dangerouslySetInnerHTML={{ __html: metric.comment }} /></Table.Cell>}
                             {!hiddenColumns.includes("issues") && <Table.Cell style={style}>
-                                <IssueStatus
-                                    metric={metric}
-                                    issueTrackerMissing={!report.issue_tracker && !report.report_uuid.startsWith("tag-")}
-                                    issueSettings={issueSettings}
-                                />
+                                <IssueStatus metric={metric} issueTrackerMissing={!report.issue_tracker} issueSettings={issueSettings} />
                             </Table.Cell>}
                             {!hiddenColumns.includes("tags") && <Table.Cell style={style}>{getMetricTags(metric).map((tag) => <Tag key={tag} tag={tag} />)}</Table.Cell>}
                         </TableRowWithDetails>
@@ -144,3 +142,24 @@ export function SubjectTable({
         </Table>
     )
 }
+SubjectTable.propTypes = {
+    changed_fields: stringsPropType,
+    dates: datesPropType,
+    handleSort: PropTypes.func,
+    hiddenColumns: stringsPropType,
+    issueSettings: issueSettingsPropType,
+    measurements: PropTypes.array,
+    metricEntries: PropTypes.array,
+    reload: PropTypes.func,
+    report: PropTypes.shape({
+        issue_tracker: PropTypes.object,
+    }),
+    reportDate: datePropType,
+    reports: reportsPropType,
+    sortDirection: sortDirectionPropType,
+    sortColumn: PropTypes.string,
+    subject: PropTypes.object,
+    subject_uuid: PropTypes.string,
+    toggleVisibleDetailsTab: PropTypes.func,
+    visibleDetailsTabs: stringsPropType
+}
diff --git a/components/frontend/src/subject/SubjectTitle.js b/components/frontend/src/subject/SubjectTitle.js
index b0beb93483..f3638e3f90 100644
--- a/components/frontend/src/subject/SubjectTitle.js
+++ b/components/frontend/src/subject/SubjectTitle.js
@@ -50,15 +50,16 @@ ButtonRow.propTypes = {
     reload: PropTypes.func
 }
 
-export function SubjectTitle({ report, subject, subject_uuid, firstSubject, lastSubject, reload }) {
+export function SubjectTitle({ atReportsOverview, report, subject, subject_uuid, firstSubject, lastSubject, reload }) {
     const dataModel = useContext(DataModel)
     const subjectType = dataModel.subjects[subject.type] || { name: "Unknown subject type" };
-    const subject_name = subject.name || subjectType.name;
+    const subjectName = subject.name || subjectType.name;
+    const subjectTitle = (atReportsOverview ? report.title + " ❯ " : "") + subjectName
     const subjectUrl = `${window.location}#${subject_uuid}`
     const panes = [
         {
             menuItem: <Menu.Item key="configuration"><Icon name="settings" /><FocusableTab>{"Configuration"}</FocusableTab></Menu.Item>,
-            render: () => <Tab.Pane><SubjectParameters subject={subject} subject_uuid={subject_uuid} subject_name={subject_name} reload={reload} /></Tab.Pane>
+            render: () => <Tab.Pane><SubjectParameters subject={subject} subject_uuid={subject_uuid} subject_name={subjectName} reload={reload} /></Tab.Pane>
         },
         {
             menuItem: <Menu.Item key="changelog"><Icon name="history" /><FocusableTab>{"Changelog"}</FocusableTab></Menu.Item>,
@@ -70,7 +71,7 @@ export function SubjectTitle({ report, subject, subject_uuid, firstSubject, last
         }
     ];
     return (
-        <HeaderWithDetails className="sticky" level="h2" header={subject_name} subheader={subject.subtitle} style={{ marginTop: 50 }}>
+        <HeaderWithDetails className="sticky" level="h2" header={subjectTitle} subheader={subject.subtitle} style={{ marginTop: 50 }}>
             <SubjectHeader subjectType={subjectType} />
             <Tab panes={panes} />
             <div style={{ marginTop: "20px" }}>
@@ -80,6 +81,7 @@ export function SubjectTitle({ report, subject, subject_uuid, firstSubject, last
     )
 }
 SubjectTitle.propTypes = {
+    atReportsOverview: PropTypes.bool,
     report: PropTypes.object,
     subject: PropTypes.object,
     subject_uuid: PropTypes.string,
diff --git a/components/frontend/src/subject/Subjects.js b/components/frontend/src/subject/Subjects.js
index 923d37dcf5..ab9d77b188 100644
--- a/components/frontend/src/subject/Subjects.js
+++ b/components/frontend/src/subject/Subjects.js
@@ -5,6 +5,7 @@ import {
     datesPropType,
     issueSettingsPropType,
     metricsToHidePropType,
+    reportsPropType,
     sortDirectionPropType,
     stringsPropType
 } from '../sharedPropTypes';
@@ -12,14 +13,15 @@ import { useDelayedRender } from '../utils';
 import { Subject } from './Subject';
 
 export function Subjects({
+    atReportsOverview,
     changed_fields,
     dates,
     handleSort,
     hiddenColumns,
     hiddenTags,
-    metricsToHide,
     issueSettings,
     measurements,
+    metricsToHide,
     reload,
     reports,
     reportsToShow,
@@ -42,6 +44,7 @@ export function Subjects({
             {subjects.map(([report, subject_uuid], index) =>
                 visible || index < 3 ?
                     <Subject
+                        atReportsOverview={atReportsOverview}
                         changed_fields={changed_fields}
                         dates={dates}
                         firstSubject={index === 0}
@@ -68,6 +71,7 @@ export function Subjects({
     )
 }
 Subjects.propTypes = {
+    atReportsOverview: PropTypes.bool,
     changed_fields: stringsPropType,
     dates: datesPropType,
     handleSort: PropTypes.func,
@@ -77,8 +81,8 @@ Subjects.propTypes = {
     measurements: PropTypes.array,
     metricsToHide: metricsToHidePropType,
     reload: PropTypes.func,
-    reports: PropTypes.array,
-    reportsToShow: PropTypes.arrayOf(PropTypes.object),
+    reports: reportsPropType,
+    reportsToShow: reportsPropType,
     report_date: datePropType,
     sortColumn: PropTypes.string,
     sortDirection: sortDirectionPropType,
diff --git a/components/frontend/src/subject/SubjectsButtonRow.js b/components/frontend/src/subject/SubjectsButtonRow.js
index 38484a1f73..f2c7f49bf4 100644
--- a/components/frontend/src/subject/SubjectsButtonRow.js
+++ b/components/frontend/src/subject/SubjectsButtonRow.js
@@ -7,6 +7,7 @@ import { AddDropdownButton, CopyButton, MoveButton } from '../widgets/Button';
 import { add_subject, copy_subject, move_subject } from '../api/subject';
 import { subject_options } from '../widgets/menu_options';
 import { subjectTypes } from './SubjectType';
+import { reportPropType, reportsPropType } from '../sharedPropTypes';
 
 export function SubjectsButtonRow({ reload, report, reports }) {
     const dataModel = useContext(DataModel)
@@ -34,6 +35,6 @@ export function SubjectsButtonRow({ reload, report, reports }) {
 }
 SubjectsButtonRow.propTypes = {
     reload: PropTypes.func,
-    report: PropTypes.object,
-    reports: PropTypes.array
+    report: reportPropType,
+    reports: reportsPropType
 }
\ No newline at end of file
diff --git a/components/frontend/src/utils.js b/components/frontend/src/utils.js
index aeec86c3e2..84754bb08e 100644
--- a/components/frontend/src/utils.js
+++ b/components/frontend/src/utils.js
@@ -358,8 +358,8 @@ export function toISODateStringInCurrentTZ(date) {
     return `${String(date.getFullYear())}-${String(date.getMonth() + 1).padStart(2, "0")}-${String(date.getDate()).padStart(2, "0")}`
 }
 
-export function getUserPermissions(username, email, current_report_is_tag_report, report_date, permissions) {
-    if (username === null || report_date !== null || current_report_is_tag_report) { return [] }
+export function getUserPermissions(username, email, report_date, permissions) {
+    if (username === null || report_date !== null) { return [] }
     return PERMISSIONS.filter((permission) => {
         const permittedUsers = permissions?.[permission] ?? [];
         return permittedUsers.length === 0 ? true : permittedUsers.includes(username) || permittedUsers.includes(email)
@@ -382,7 +382,3 @@ export function sum(object) {
     const list = typeof object == Array ? object : Object.values(object)
     return list.reduce((a, b) => a + b, 0)
 }
-
-export function reportIsTagReport(report_uuid) {
-    return report_uuid.slice(0, 4) === "tag-"
-}
diff --git a/components/frontend/src/utils.test.js b/components/frontend/src/utils.test.js
index 9a7f65672d..97e8aff0d5 100644
--- a/components/frontend/src/utils.test.js
+++ b/components/frontend/src/utils.test.js
@@ -3,7 +3,7 @@ import history from 'history/browser';
 import {
     capitalize, getUserPermissions, getMetricTags, get_metric_target, getReportTags, get_source_name, get_subject_name,
     nice_number, scaled_number, registeredURLSearchParams, userPrefersDarkMode, useURLSearchQuery,
-    getMetricResponseOverrun, reportIsTagReport, visibleMetrics
+    getMetricResponseOverrun, visibleMetrics
 } from './utils';
 import { EDIT_REPORT_PERMISSION, EDIT_ENTITY_PERMISSION } from './context/Permissions';
 
@@ -63,32 +63,27 @@ it('adds a scale', () => {
 });
 
 it('gives users all permissions if permissions have not been limited', () => {
-    const permissions = getUserPermissions("jodoe", "john.doe@example.org", false, null, {});
+    const permissions = getUserPermissions("jodoe", "john.doe@example.org", null, {});
     expect(permissions).toStrictEqual([EDIT_REPORT_PERMISSION, EDIT_ENTITY_PERMISSION]);
 });
 
 it('gives users edit report permissions if edit report permissions have been granted', () => {
-    const permissions = getUserPermissions("jodoe", "john.doe@example.org", false, null, { [EDIT_REPORT_PERMISSION]: ["jodoe"], [EDIT_ENTITY_PERMISSION]: ["jadoe"] });
+    const permissions = getUserPermissions("jodoe", "john.doe@example.org", null, { [EDIT_REPORT_PERMISSION]: ["jodoe"], [EDIT_ENTITY_PERMISSION]: ["jadoe"] });
     expect(permissions).toStrictEqual([EDIT_REPORT_PERMISSION]);
 });
 
 it('gives users edit entity permissions if edit entity permissions have been granted', () => {
-    const permissions = getUserPermissions("jodoe", "john.doe@example.org", false, null, { [EDIT_REPORT_PERMISSION]: ["jadoe"], [EDIT_ENTITY_PERMISSION]: ["jodoe"] });
+    const permissions = getUserPermissions("jodoe", "john.doe@example.org", null, { [EDIT_REPORT_PERMISSION]: ["jadoe"], [EDIT_ENTITY_PERMISSION]: ["jodoe"] });
     expect(permissions).toStrictEqual([EDIT_ENTITY_PERMISSION]);
 });
 
 it('gives users no permissions if they have not logged in', () => {
-    const permissions = getUserPermissions(null, null, false, null, {});
+    const permissions = getUserPermissions(null, null, null, {});
     expect(permissions).toStrictEqual([]);
 });
 
 it('gives users no permissions if the report date is in the past', () => {
-    const permissions = getUserPermissions("jodoe", "john.doe@example.org", false, new Date(), {});
-    expect(permissions).toStrictEqual([]);
-});
-
-it('gives users no permissions if the report is a tag report', () => {
-    const permissions = getUserPermissions("jodoe", "john.doe@example.org", true, null, {});
+    const permissions = getUserPermissions("jodoe", "john.doe@example.org", new Date(), {});
     expect(permissions).toStrictEqual([]);
 });
 
@@ -366,11 +361,6 @@ it("returns the metric response overrun when there are two measurements with dif
     expect(getMetricResponseOverrun("uuid", {}, {}, measurements)).toStrictEqual({"overruns": [], "totalOverrun": 0})
 })
 
-it("returns whether a report is a tag report", () => {
-    expect(reportIsTagReport("report")).toBe(false)
-    expect(reportIsTagReport("tag-report")).toBe(true)
-})
-
 it("returns the tags of an empty report", () => {
     expect(getReportTags({"subjects": {}})).toStrictEqual([])
 })
diff --git a/components/shared_code/src/shared/model/report.py b/components/shared_code/src/shared/model/report.py
index 2ecc0080b6..229ffc1f7b 100644
--- a/components/shared_code/src/shared/model/report.py
+++ b/components/shared_code/src/shared/model/report.py
@@ -41,7 +41,7 @@ def __init__(self, data_model: dict, report_data: dict) -> None:
         self.sources_dict = self._sources()
         self.sources = list(self.sources_dict.values())
 
-        if "_id" in self:
+        if "_id" in self:  # pragma: no feature-test-cover
             self["_id"] = str(self["_id"])
 
     @property
@@ -75,14 +75,10 @@ def __eq__(self, other: object) -> bool:
 
     def _subjects(self, subject_data: dict) -> dict[str, Subject]:
         """Instantiate subjects of this report."""
-        subjects = {}
-        for subject_uuid, subject in subject_data.items():
-            if isinstance(subject, Subject):
-                subjects[subject_uuid] = subject
-            else:
-                subjects[subject_uuid] = Subject(self.__data_model, subject, subject_uuid, self)
-
-        return subjects
+        return {
+            subject_uuid: Subject(self.__data_model, subject, subject_uuid, self)
+            for subject_uuid, subject in subject_data.items()
+        }
 
     def _metrics(self) -> dict[MetricId, Metric]:
         """All metrics of all subjects of this report."""
diff --git a/components/shared_code/src/shared/model/subject.py b/components/shared_code/src/shared/model/subject.py
index 28697bbd2f..25ac484cbd 100644
--- a/components/shared_code/src/shared/model/subject.py
+++ b/components/shared_code/src/shared/model/subject.py
@@ -1,6 +1,6 @@
 """A class that represents a subject."""
 
-from typing import TYPE_CHECKING, Optional, cast
+from typing import TYPE_CHECKING, cast
 
 from shared.utils.type import MetricId, SubjectId
 
@@ -56,16 +56,6 @@ def name(self) -> str | None:
         default_name = self.__data_model["subjects"].get(self.type, {}).get("name")
         return str(default_name) if default_name else None
 
-    def tag_subject(self, tag: str) -> Optional["Subject"]:
-        """Return a Subject instance with only metrics belonging to one tag."""
-        metrics = {metric.uuid: metric for metric in self.metrics if tag in metric.get("tags", [])}
-        if len(metrics) == 0:
-            return None
-        data = dict(self)
-        data["metrics"] = metrics
-        data["name"] = self.report.name + " ❯ " + (self.name or "")  # noqa: RUF001
-        return Subject(self.__data_model, data, self.uuid, self.report)
-
     def summarize(self, measurements: dict[MetricId, list[Measurement]]) -> dict:
         """Create a summary dict of this subject."""
         summary = dict(self)
diff --git a/components/shared_code/tests/shared/model/test_subject.py b/components/shared_code/tests/shared/model/test_subject.py
index a031601e3e..58177ddd37 100644
--- a/components/shared_code/tests/shared/model/test_subject.py
+++ b/components/shared_code/tests/shared/model/test_subject.py
@@ -54,30 +54,6 @@ def test_missing_default_name(self):
         subject = Subject(data_model, {"type": "software"}, SUBJECT_ID, self.report)
         self.assertIsNone(subject.name)
 
-    def test_tag_subject(self):
-        """Test that a new subject can be created with metrics filtered by tag."""
-        metric = Metric(self.DATA_MODEL, {"tags": ["tag"]}, METRIC_ID)
-        subject = Subject(
-            self.DATA_MODEL,
-            {"name": "Subject", "metrics": {METRIC_ID: metric}},
-            SUBJECT_ID,
-            self.report,
-        )
-        tag_subject = subject.tag_subject("tag")
-        self.assertEqual([metric], tag_subject.metrics)
-        self.assertEqual("Report ❯ Subject", tag_subject.name)  # noqa: RUF001
-
-    def test_tag_subject_with_unused_tag(self):
-        """Test that filtering a subject's metrics by a non-existing tag returns None."""
-        metric = Metric(self.DATA_MODEL, {"tags": ["tag"]}, METRIC_ID)
-        subject = Subject(
-            self.DATA_MODEL,
-            {"name": "Subject", "metrics": {METRIC_ID: metric}},
-            SUBJECT_ID,
-            self.report,
-        )
-        self.assertIsNone(subject.tag_subject("unused"))
-
     def test_summarize(self):
         """Test that a subject can be summarized."""
         metric = Metric(self.DATA_MODEL, {"type": "violations"}, METRIC_ID)
diff --git a/docs/src/changelog.md b/docs/src/changelog.md
index e33f2120b2..b45d96ec6a 100644
--- a/docs/src/changelog.md
+++ b/docs/src/changelog.md
@@ -20,6 +20,7 @@ If your currently installed *Quality-time* version is v4.10.0 or older, please r
 - Support downloading Harbor JSON vulnerability reports from a JSON file. Closes [#7147](https://github.com/ICTU/quality-time/issues/7147).
 - In addition to hiding metrics that do not require immediate action, also allow for hiding all metrics so that only the dashboard of a report is visible. Closes [#7211](https://github.com/ICTU/quality-time/issues/7211).
 - Allow for changing and resetting settings per report. Closes [#7213](https://github.com/ICTU/quality-time/issues/7213).
+- Allow for showing all metrics on the reports overview page. Closes [#7215](https://github.com/ICTU/quality-time/issues/7215).
 
 ### Changed
 
diff --git a/tests/feature_tests/src/features/tagreport.feature b/tests/feature_tests/src/features/tagreport.feature
deleted file mode 100644
index bccd3eccc5..0000000000
--- a/tests/feature_tests/src/features/tagreport.feature
+++ /dev/null
@@ -1,17 +0,0 @@
-Feature: tag reports
-  Tag reports are read-only reports that contain all metrics from all reports with a specific tag
-
-  Background: the client is logged in
-    Given a logged-in client
-    And an existing report
-    And an existing subject
-    And an existing metric
-
-  Scenario: tag reports for unused tags are empty
-    When the client gets the tag report for the tag "unused_tag"
-    Then the tag report is empty
-
-  Scenario: tag reports only contain metrics with the specified tag
-    Given an existing metric with tags "tag_report_test"
-    When the client gets the tag report for the tag "tag_report_test"
-    Then the tag report with tag "tag_report_test" has only metrics with said tag
diff --git a/tests/feature_tests/src/steps/tagreport.py b/tests/feature_tests/src/steps/tagreport.py
deleted file mode 100644
index 1e644e0206..0000000000
--- a/tests/feature_tests/src/steps/tagreport.py
+++ /dev/null
@@ -1,25 +0,0 @@
-"""Step implementations for tag reports."""
-
-from asserts import assert_equal, assert_true
-from behave import then, when
-from behave.runner import Context
-
-
-@when('the client gets the tag report for the tag "{tag}"')
-def get_tag_report(context: Context, tag: str) -> None:
-    """Get the tag report for the tag."""
-    context.reports = context.get(f"report/tag-{tag}")
-
-
-@then("the tag report is empty")
-def check_tag_report_is_empty(context: Context) -> None:
-    """Check that the tag report is empty."""
-    assert_equal(0, len(context.reports["reports"]))
-
-
-@then('the tag report with tag "{tag}" has only metrics with said tag')
-def check_tag_report(context: Context, tag: str) -> None:
-    """Check that the tag report has the expected contents."""
-    for subject in context.reports["reports"][0]["subjects"].values():
-        for metric in subject["metrics"].values():
-            assert_true(tag in metric["tags"])