Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow for showing all metrics on the reports overview page #7308

Merged
merged 1 commit into from
Oct 24, 2023
Merged

Allow for showing all metrics on the reports overview page #7308

merged 1 commit into from
Oct 24, 2023

Conversation

fniessink
Copy link
Member

@fniessink fniessink commented Oct 18, 2023 •
edited
Loading

Closes #7214 and closes #7215 and closes #1301.

@fniessink fniessink linked an issue Oct 18, 2023 that may be closed by this pull request
Allow for showing all metrics on the report overview page #7215
Closed
14 tasks
@fniessink fniessink changed the title Allow for showing all metrics on the reports overview page. Closes #7… Allow for showing all metrics on the reports overview page Oct 18, 2023
@fniessink fniessink force-pushed the 7215-allow-for-showing-all-metrics-on-the-report-overview-page branch 12 times, most recently from b3f6e9b to 9d72900 Compare October 20, 2023 13:57
@fniessink fniessink marked this pull request as ready for review October 20, 2023 13:59
@fniessink fniessink force-pushed the 7215-allow-for-showing-all-metrics-on-the-report-overview-page branch 7 times, most recently from ced5641 to 6b4673a Compare October 21, 2023 19:28
@fniessink fniessink requested a review from wkoot October 22, 2023 18:44
wkoot
wkoot reviewed Oct 23, 2023
View reviewed changes
docs/src/changelog.md Outdated Show resolved Hide resolved
components/api_server/src/routes/pdf.py Show resolved Hide resolved
components/frontend/src/AppUI.js Outdated Show resolved Hide resolved
components/frontend/src/header_footer/SettingsPanel.js Show resolved Hide resolved
components/frontend/src/report/Report.js Show resolved Hide resolved
components/frontend/src/report/ReportsOverview.test.js Show resolved Hide resolved
components/frontend/src/subject/Subjects.js Outdated Show resolved Hide resolved
components/frontend/src/subject/SubjectsButtonRow.js Outdated Show resolved Hide resolved
components/frontend/src/widgets/Button.js Outdated Show resolved Hide resolved
components/frontend/src/widgets/Button.js Outdated Show resolved Hide resolved
@fniessink fniessink force-pushed the 7215-allow-for-showing-all-metrics-on-the-report-overview-page branch from 6b4673a to e279c12 Compare October 24, 2023 10:00
@fniessink fniessink force-pushed the 7215-allow-for-showing-all-metrics-on-the-report-overview-page branch from e279c12 to 26ef487 Compare October 24, 2023 10:50
@sonarcloud
Copy link

sonarcloud bot commented Oct 24, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

95.8% 95.8% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@fniessink fniessink merged commit 153e35c into master Oct 24, 2023
37 of 38 checks passed
@fniessink fniessink deleted the 7215-allow-for-showing-all-metrics-on-the-report-overview-page branch October 24, 2023 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wkoot wkoot wkoot left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@fniessink @wkoot