Conversation
Filter-driven reloads are now centralized to prevent multiple data fetches at once. Further, added a feature to cancel ongoing requests and ensure that older responses don’t replace newer data. After applying filters, there’s a temporary pause on auto-refresh to prevent interruptions while users are still filtering.
Removed duplicate fetch behavior in search/filter apply logic by avoiding page-navigation fetch plus manual reload in the same interaction.
geichelberger
force-pushed
the
less-queries
branch
from
6b979f9 to
3146682
Compare
|
Use Run test server using develop.opencast.org as backend: Specify a different backend like stable.opencast.org: It may take a few seconds for the interface to spin up. |
|
This pull request is deployed at test.admin-interface.opencast.org/1542/2026-04-03_14-43-49/ . |
|
This pull request has conflicts ☹ |
Arnei
left a comment
Arnei
left a comment
There was a problem hiding this comment.
Reads sound to me and works from my testing. I would like to see a brief comment in the code that explains the intent behind these functions (basically what you already wrote in the commit message).
| dispatch(goToPage(0)); | ||
| dispatch(deselectAll()); | ||
| dispatch(setOffset(0)); | ||
| dispatch(setPageActive(1)); |
There was a problem hiding this comment.
Should this not set to 0 instead of 1?
|
|
||
| currentLoadRequest.current?.abort?.(); | ||
|
|
||
| const fetchRequest = dispatch(fetchResource()) as Promise<PayloadAction<any, string>> & { abort:() => void }; |
There was a problem hiding this comment.
I think we can do without the type assertions here to make this a bit easier to read?
2 participants
Filter-driven reloads are now centralized to prevent multiple data
fetches at once. Further, added a feature to cancel ongoing requests and
ensure that older responses don’t replace newer data.
After applying filters, there’s a temporary pause on auto-refresh to
prevent interruptions while users are still filtering.
Removed duplicate fetch behavior in search/filter apply logic by
avoiding page-navigation fetch plus manual reload in the same
interaction.