Closed hodanoori closed 2 weeks ago
Latest commit: 5c8e01afedac0f1e8dd187186a8b6c49cff1bca7
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
PR Preview Action v1.4.8 :---: Preview removed because the pull request was closed. 2024-11-15 10:44 UTC
Hi! I started checking the preview, and here’s what I noticed so far: the current page in pagination isn't being saved in the URL. For example, if I'm on page 3/3 in the Services tab and click on a service, the details panel opens correctly. However, when I reload the page, while the panel is correctly populated, the pagination resets to page 1/3, breaking the connection to the details shown in the panel. Could this behavior be related to the recent improvements?
Hi Arturo, nice catch, yes, the pagination is not correctly kept in the URL. But I will add the check for pagination and search in url state in a separate PR, if it's fine for you.
I'm experiencing the same issue with the search term. It seems that both pagination and the search term are not being reflected in the URL, correct?
Correct, I fix the search along with pagination in next PR.
By clicking I have noticed that the description of an IssueMatch in the details panel stays in Loading state for ever. Is this an Issue and do you want to create a ticket for it?
This is known to me, I fix this issue also in a separate PR.
Summary
Fixed an issue where activeFilters were not immediately applied after URL updates, requiring additional actions to trigger synchronization. Refactored useUrlState by updating useEffect dependencies to ensure real-time syncing of URL state and filters when changes occur. Additionally, restored the missing activeFilters prop in FilterSelect within the Filters control.
Furthermore, the detailsFor prop is now handled correctly, ensuring that when a detail panel is closed, the corresponding detailsFor information is removed from the URL, maintaining a clean and accurate URL state.
Changes Made
Related Issues
Testing Instructions
For example, the following part of URL should display the
Support group ccrn = compute-storage-api
filter on theServices
view: f:(Components:(:),IssueMatches:(:),Services:(supportGroupCcrn:(compute~Fstorage~Fapi)))npm i
npm run TASK
Checklist