Skip to content

[O2B-1534] Migrate log overview to use filtering model pattern#2083

Open
NarrowsProjects wants to merge 55 commits intomainfrom
improv/O2B-1534/Migrate-Log-Overview-to-use-FilteringModel-pattern
Open

[O2B-1534] Migrate log overview to use filtering model pattern#2083
NarrowsProjects wants to merge 55 commits intomainfrom
improv/O2B-1534/Migrate-Log-Overview-to-use-FilteringModel-pattern

Conversation

@NarrowsProjects
Copy link
Copy Markdown
Collaborator

@NarrowsProjects NarrowsProjects commented Feb 20, 2026

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:

  • Individual from-to selectors on the logs page have been replaced with a single field that will open the standard time range selector
  • All open-input forms apply their filters when focus is lost on them.

Notable changes for developers:

  • All filters have been moved to a filteringModel in logsOverviewModel
  • The filters for runs, lhcfills and environments have been renamed under the hood to match getAllRunsUseCase:
    • runs => runNumbers
    • lhcFills => fillNumbers
    • environments => environmentIds

Changes made to the database:

  • none

@NarrowsProjects NarrowsProjects self-assigned this Feb 20, 2026
@NarrowsProjects NarrowsProjects added frontend javascript Pull requests that update Javascript code labels Feb 20, 2026
@NarrowsProjects NarrowsProjects marked this pull request as draft February 20, 2026 11:53
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 20, 2026

Codecov Report

❌ Patch coverage is 34.78261% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.76%. Comparing base (1333378) to head (4fa1173).

Files with missing lines Patch % Lines
...ib/public/views/Logs/Overview/LogsOverviewModel.js 0.00% 11 Missing ⚠️
...blic/views/Logs/ActiveColumns/logsActiveColumns.js 0.00% 8 Missing ⚠️
...nts/Filters/LogsFilter/author/AuthorFilterModel.js 0.00% 6 Missing ⚠️
...mponents/Filters/LogsFilter/author/authorFilter.js 0.00% 3 Missing ⚠️
...ic/components/Filters/common/filters/textFilter.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2083      +/-   ##
==========================================
+ Coverage   45.50%   45.76%   +0.25%     
==========================================
  Files        1044     1042       -2     
  Lines       17353    17213     -140     
  Branches     3149     3119      -30     
==========================================
- Hits         7897     7877      -20     
+ Misses       9456     9336     -120     

☔ 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.

@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1534/Migrate-Log-Overview-to-use-FilteringModel-pattern branch from 07567e2 to 114285d Compare February 20, 2026 12:02
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1534/Migrate-Log-Overview-to-use-FilteringModel-pattern branch from a936a03 to 5500b3e Compare February 20, 2026 16:24
@NarrowsProjects NarrowsProjects marked this pull request as ready for review February 21, 2026 18:22
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1534/Migrate-Log-Overview-to-use-FilteringModel-pattern branch from 3cd62d0 to 370c475 Compare April 13, 2026 13:19
@@ -233,7 +233,7 @@ module.exports = () => {
});

Copy link
Copy Markdown
Collaborator

@isaachilly isaachilly Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorting by runs nor environments is tested as well as filtering by rootOnly. Would be good I think to take this chance to add these tests now.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Taken care off

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend javascript Pull requests that update Javascript code

Development

Successfully merging this pull request may close these issues.

2 participants