Skip to content

Add server-side sort, filter, and pagination to changelog#794

Merged
garrettjstevens merged 2 commits into
mainfrom
change_endpoint_queries
May 20, 2026
Merged

Add server-side sort, filter, and pagination to changelog#794
garrettjstevens merged 2 commits into
mainfrom
change_endpoint_queries

Conversation

@garrettjstevens
Copy link
Copy Markdown
Contributor

This adds server-side sorting, filtering, and pagination to GET requests to the changes endpoint. This is then used to power the DataGrid from @mui/x-data-grid, making the UI much more responsive when there are many changes.

The changelog also still works for local editing when the changes are stored in indexedDb.

@garrettjstevens
Copy link
Copy Markdown
Contributor Author

@shashankbrgowda This is a proposed alternative to #793. I wanted to make sure any filtering and pagination we added would also work well with the changelog component in the UI. Datetime filtering and pagination is still available in this PR, just implemented a bit differently to work with the DataGrid component. Here's an example request:

http://localhost:3999/changes?assembly=697d2a056cf6618c23df2bb2&page=0&pageSize=15&sortField=sequence&sortOrder=desc&startTime=2026-05-12T06%3A00%3A00.000Z

Let me know if this works for your use case.

@garrettjstevens garrettjstevens merged commit 667158f into main May 20, 2026
7 checks passed
@garrettjstevens garrettjstevens deleted the change_endpoint_queries branch May 20, 2026 17:51
@shashankbrgowda
Copy link
Copy Markdown
Contributor

@shashankbrgowda This is a proposed alternative to #793. I wanted to make sure any filtering and pagination we added would also work well with the changelog component in the UI. Datetime filtering and pagination is still available in this PR, just implemented a bit differently to work with the DataGrid component. Here's an example request:

http://localhost:3999/changes?assembly=697d2a056cf6618c23df2bb2&page=0&pageSize=15&sortField=sequence&sortOrder=desc&startTime=2026-05-12T06%3A00%3A00.000Z

Let me know if this works for your use case.

Thanks @garrettjstevens . This should work fine

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.

2 participants