Skip to content

add: activity manager for async export report - #5729

Merged
greenbonebot merged 2 commits into
mainfrom
gea-2092-2
Oct 20, 2026
Merged

greenbonebot merged 2 commits into
mainfrom
gea-2092-2

Conversation

@daniele-mng

@daniele-mng daniele-mng commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

What

  • Support async export reports with activity manager

Why

References

GEA-2092

Checklist

  • I have added tests for the changes
  • I have used the following LLMs/AI tools in this pull request:
Screencast.From.2026-10-20.08-01-02.webm

@github-actions

github-actions Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 3ed6293.
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.

Scanned Files

None

@daniele-mng
daniele-mng force-pushed the gea-2092-2 branch 2 times, most recently from 539a2fa to 3e51b1a Compare October 20, 2026 06:13
@codecov

codecov Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.05582% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.88%. Comparing base (83ec3cf) to head (3ed6293).

Files with missing lines Patch % Lines
.../web/pages/reports/AuditDeltaReportDetailsPage.jsx 42.85% 6 Missing and 2 partials ⚠️
src/web/report-export/useReportExports.ts 96.51% 6 Missing and 1 partial ⚠️
src/web/report-export/report-export-job-display.ts 90.56% 5 Missing ⚠️
src/web/report-export/ReportExportManager.tsx 89.47% 4 Missing ⚠️
src/web/report-export/report-export-job-model.ts 97.27% 3 Missing ⚠️
src/web/pages/Page.tsx 50.00% 1 Missing ⚠️
src/web/pages/reports/DownloadReportDialog.tsx 83.33% 1 Missing ⚠️
src/web/report-export/ReportExportJobRow.tsx 97.56% 1 Missing ⚠️
src/web/report-export/useReportExportQueries.ts 97.22% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5729      +/-   ##
==========================================
+ Coverage   87.84%   87.88%   +0.04%     
==========================================
  Files        1328     1341      +13     
  Lines       34636    35390     +754     
  Branches    10346    10624     +278     
==========================================
+ Hits        30425    31104     +679     
- Misses       3811     3878      +67     
- Partials      400      408       +8     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@daniele-mng
daniele-mng marked this pull request as ready for review October 20, 2026 09:06
@daniele-mng
daniele-mng requested a review from a team as a code owner October 20, 2026 09:06
@greenbonebot
greenbonebot enabled auto-merge (rebase) October 20, 2026 09:06
Comment thread src/web/report-export/report-export-job-model.ts Outdated
Comment thread src/gmp/commands/report-export.ts Outdated
@daniele-mng
daniele-mng requested a review from ozgen October 20, 2026 12:11
@greenbonebot
greenbonebot merged commit 9f05e53 into main Oct 20, 2026
28 checks passed
@greenbonebot
greenbonebot deleted the gea-2092-2 branch October 20, 2026 12:21
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