Skip to content

fix: prevent redirect when navigating to retrieval-check#2439

Merged
lidel merged 1 commit intomainfrom
fix/diagnostics-retrieval-check-navigation
Sep 26, 2025
Merged

fix: prevent redirect when navigating to retrieval-check#2439
lidel merged 1 commit intomainfrom
fix/diagnostics-retrieval-check-navigation

Conversation

@lidel
Copy link
Member

@lidel lidel commented Sep 19, 2025

The diagnostics page was redirecting to /logs when clicking "Check Retrieval" from the Files context menu. This happened because the redirect logic fired too early, before the full route was available.

Fixed by checking the complete URL path before redirecting.

@lidel lidel requested a review from a team as a code owner September 19, 2025 22:55
@lidel lidel mentioned this pull request Sep 19, 2025
49 tasks
The diagnostics page was redirecting to /logs when clicking "Check Retrieval"
from the Files context menu. This happened because the redirect logic fired
too early, before the full route was available.

Fixed by checking the complete URL path before redirecting.
@lidel lidel force-pushed the fix/diagnostics-retrieval-check-navigation branch from cfa317a to 9d870cd Compare September 26, 2025 01:56
@lidel lidel merged commit bd11b0c into main Sep 26, 2025
23 checks passed
@lidel lidel deleted the fix/diagnostics-retrieval-check-navigation branch September 26, 2025 02:06
ipfs-gui-bot pushed a commit that referenced this pull request Sep 26, 2025
## [4.9.1](v4.9.0...v4.9.1) (2025-09-26)

 CID `bafybeicg7e6o2eszkfdzxg5233gmuip2a7kfzoloh7voyvt2r6ivdet54u`

 ---

### Bug Fixes

* flash when flipping to/from previewing item ([#2437](#2437)) ([c14b38a](c14b38a))
* overlay click does not close modals ([#2428](#2428)) ([f239113](f239113))
* prevent redirect when navigating to retrieval-check ([#2439](#2439)) ([bd11b0c](bd11b0c))
@ipfs-gui-bot
Copy link
Collaborator

🎉 This PR is included in version 4.9.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants