Skip to content

[O2B-1588] Implement summary views for overview pages#2156

Open
NarrowsProjects wants to merge 158 commits into
mainfrom
improv/O2B-1588/implement-summary-views-for-overview-pages
Open

[O2B-1588] Implement summary views for overview pages#2156
NarrowsProjects wants to merge 158 commits into
mainfrom
improv/O2B-1588/implement-summary-views-for-overview-pages

Conversation

@NarrowsProjects

@NarrowsProjects NarrowsProjects commented May 13, 2026 •
edited
Loading

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:

  • N/A

Notable changes for developers:

  • Summary views have been added for listAll endpoints (services and GetAllUseCase files) of:

    • Logs
    • QcFlagTypes
    • LhcFills
    • Environments
    • Runs
    • DataPasses

  • IMPORTANT:

    • QueryBuilder has been rewritten to use attributes array syntax instead of the previous object syntax.
      • This is required for limiting what top-level fields should be excluded
      • The behavior of QueryBuilder has been rewritten to mostly mimic the behavior of object syntax by manually adding all columns of a model if no specific columns have been provided. Computed columns are not included for this
      • If you want to include only computed columns without fetching all other columns, you can use the new function called selectOnly

Changes made to the database:

  • N/A

Guust and others added 30 commits February 20, 2026 10:23
chore: remove redundant model assignment.
722caf7
chore: move all filters denoted as so to a filteringModel
f161cae
Make filterInputModel extend filterModel
9faffaf
rename inputfilter to ParsedInputFilter
247943b
implement authorfilter with ParsedInputFilterModel changes
114285d
re-implement filter
f39f7ae
add tag-filters to the filtering object
df1fea3
replace titleFilter and contentFilter with RawTextFilterModels
96ca85f
make Authorfilter an implementation of RawTextFilterModel
93b3291
add runs filter to the filteringmodel
5a5e830
add environments filter to the filteringmodel
b5e7148
add lhcFills filter to the filteringmodel
3507ba0
add created filter to the filteringmodel
9c63409
move the sort 'filter' to fetchlogs, since it doesn't actually filter…
5500b3e 
… anything
change filter to rawTextFilter for title
a09ae8b
fix test by changing event type to 'change'
512999b
fix content and author tests
0f15812
import openFilteringPanel and resetFilters
7b641ca
fix createdAt filter test
5ab8218
change event types to change
4c96d73
fix isAnyFilterActive
18ce4dc
remove _raw as from authorfilterModel, as it serves no purpose
685dd38
chore: removed ParsedInputFilterModel
4e132fd
chore: add toLowerCase to filterQueryParam computation
caf9812
@NarrowsProjects
[O2B-1530] Lhc fills add sb duration filter (#2080)
41c5a72 
* Filtering by stableBeamsStart and stableBeamsEnd has been added to LHC Fills overview page
* lhcFills endpoint & DTO validation modified and testing added for the aforementioned changes
---------

Co-authored-by: GuustMetz <guust.metz@cern.ch>
Co-authored-by: Guust <metzguust@gmail.com>
@isaachilly
[O2B-1544] Fix pagination for filtered envs and add a test (#2096)
f38f597 
* Replaced the two-query pattern with a single queryBuilder in GetAllEnvironmentsUseCase. The previous approach was redundant following Sequelize performance improvements; furthermore, the original implementation's logic was flawed which resulted in the pagination bug.
remove the combination operator from runs
3efdf1d
remove the combination operator from envirionments
fc61bea
remove the combination operator from lhcFills
9c4df68
make filter computation much more compact using filteringmodel.normalize
b860320
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1588/implement-summary-views-for-overview-pages branch 13 times, most recently from 7a14c8d to 8cf3cb6 Compare May 17, 2026 20:55
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1588/implement-summary-views-for-overview-pages branch from 8cf3cb6 to 49cfbdc Compare May 17, 2026 21:00
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1588/implement-summary-views-for-overview-pages branch 3 times, most recently from bdfb1bf to 1a14371 Compare May 17, 2026 22:31
@NarrowsProjects NarrowsProjects force-pushed the improv/O2B-1588/implement-summary-views-for-overview-pages branch from 4239c0a to 1813f6c Compare May 17, 2026 22:48
@NarrowsProjects NarrowsProjects force-pushed the feature/O2B-1560/Add-filter-is-active-indicator-to-the-UI branch from 31a3b32 to 505156d Compare June 18, 2026 10:10
Base automatically changed from feature/O2B-1560/Add-filter-is-active-indicator-to-the-UI to main June 18, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@isaachilly isaachilly Awaiting requested review from isaachilly isaachilly is a code owner

@graduta graduta Awaiting requested review from graduta

At least 1 approving review is required to merge this pull request.

Assignees

@NarrowsProjects NarrowsProjects

Labels

backend javascript Pull requests that update Javascript code

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NarrowsProjects @isaachilly