Skip to content

[O2B-1293] Runs API extension for QC flags export#1948

Closed
xsalonx wants to merge 22 commits into
mainfrom
xsalonx/export/O2B-1293/qc-flags-export
Closed

[O2B-1293] Runs API extension for QC flags export#1948
xsalonx wants to merge 22 commits into
mainfrom
xsalonx/export/O2B-1293/qc-flags-export

Conversation

@xsalonx

@xsalonx xsalonx commented Jul 31, 2025

Copy link
Copy Markdown
Collaborator

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • NA

Notable changes for developers:

  • Allow to include QC flags in get-all-runs request

Changes made to the database:

  • NA

@codecov

codecov Bot commented Jul 31, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 4.16667% with 161 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.38%. Comparing base (64a1438) to head (b1b15da).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
lib/public/models/DataExportModel.js 0.00% 59 Missing ⚠️
...ublic/views/Runs/Overview/exportTriggerAndModal.js 0.00% 43 Missing ⚠️
lib/public/models/OverviewModel.js 0.00% 28 Missing ⚠️
lib/usecases/run/GetAllRunsUseCase.js 16.66% 15 Missing ⚠️
...ib/public/views/Runs/Overview/RunsOverviewModel.js 0.00% 7 Missing ⚠️
lib/domain/dtos/GetAllRunsDto.js 0.00% 6 Missing ⚠️
lib/database/adapters/RunAdapter.js 50.00% 3 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (64a1438) and HEAD (b1b15da). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (64a1438) HEAD (b1b15da)
15 14
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1948      +/-   ##
==========================================
- Coverage   45.59%   37.38%   -8.22%     
==========================================
  Files        1014     1015       +1     
  Lines       16699    16793      +94     
  Branches     3006     3022      +16     
==========================================
- Hits         7614     6278    -1336     
- Misses       9085    10515    +1430     

☔ View full report in Codecov by Sentry.
📢 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.

@xsalonx xsalonx changed the title Xsalonx/export/o2 b 1293/qc flags export [O2B-1293] QC flags export Aug 1, 2025
@xsalonx xsalonx changed the title [O2B-1293] QC flags export [O2B-1293] Runs API extension for QC flags export Aug 1, 2025
@xsalonx xsalonx removed the frontend label Aug 1, 2025
@xsalonx

xsalonx commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator Author

To be merged after #1947

@xsalonx xsalonx marked this pull request as ready for review August 1, 2025 10:25
@xsalonx xsalonx requested a review from graduta as a code owner August 1, 2025 10:25
@xsalonx xsalonx closed this Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant