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.

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

Removed feature toggle for compliance report views. #4187

Conversation

jhelmold
Copy link
Member

What

Removed the feature toggle for the dedicated compliance report views.

Why

All feature toggles are being removed for the new major release 24.10.

References

GEA-712

Checklist

  • Tests

Copy link

github-actions bot commented Oct 28, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 15ef6bd.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

github-actions bot commented Oct 28, 2024

Conventional Commits Report

Type Number
Removed 1

🚀 Conventional commits found.

Removed the feature toggle for the dedicated compliance report views.
@jhelmold jhelmold force-pushed the GEA-712_Remove_feature_toggle_COMPLIANCE_REPORTS_opensight branch from 684ee43 to 0ec8956 Compare October 28, 2024 09:42
Copy link

codecov bot commented Oct 28, 2024

Codecov Report

Attention: Patch coverage is 20.68966% with 23 lines in your changes missing coverage. Please review.

Project coverage is 64.38%. Comparing base (bfbd85a) to head (15ef6bd).
Report is 4 commits behind head on opensight-ui.

Files with missing lines Patch % Lines
src/web/pages/usersettings/filterpart.jsx 0.00% 10 Missing ⚠️
src/web/pages/usersettings/usersettingspage.jsx 0.00% 6 Missing ⚠️
src/web/routes.jsx 0.00% 4 Missing ⚠️
src/web/pages/filters/component.jsx 0.00% 2 Missing ⚠️
src/web/pages/tags/component.jsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           opensight-ui    #4187      +/-   ##
================================================
+ Coverage         64.35%   64.38%   +0.02%     
================================================
  Files              1111     1111              
  Lines            103977   103926      -51     
  Branches           6496     6493       -3     
================================================
- Hits              66917    66908       -9     
+ Misses            36921    36879      -42     
  Partials            139      139              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jhelmold jhelmold marked this pull request as ready for review October 28, 2024 11:00
@jhelmold jhelmold requested a review from a team as a code owner October 28, 2024 11:00
@daniele-mng
Copy link
Contributor

Quick note: if the changes are also for v24, we should open a second PR against branch opensight-ui.

@a-h-abdelsalam
Copy link
Contributor

@daniele-mng yes I think the changes are only needed for 24.10 and so the PR is only against opensight-ui.

@a-h-abdelsalam a-h-abdelsalam merged commit f355cbf into opensight-ui Nov 7, 2024
9 of 10 checks passed
@a-h-abdelsalam a-h-abdelsalam deleted the GEA-712_Remove_feature_toggle_COMPLIANCE_REPORTS_opensight branch November 7, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants