Skip to content

feat: add useExtensionDetail hook for managing extension details state#1760

Merged
netomi merged 7 commits intoeclipse-openvsx:mainfrom
gnugomez:gnugomez/main/fix-extension-details-state
Apr 15, 2026
Merged

feat: add useExtensionDetail hook for managing extension details state#1760
netomi merged 7 commits intoeclipse-openvsx:mainfrom
gnugomez:gnugomez/main/fix-extension-details-state

Conversation

@gnugomez
Copy link
Copy Markdown
Member

@gnugomez gnugomez commented Apr 14, 2026

Resolves #1758 #1761

This PR is resolving two bugs.

  • The extension details request to the api was triggered twice, the first load was triggering the two useEffects
  • When a different version was selected, the extension readme wasn't refreshed, you would need to trigger a full reload

I'm taking the opportunity to re-architecture this component a bit, I'm extracting the state logic into a separate custom hook, using the Route component for tabs for clarity.

Assisted-By: github:copilot

@gnugomez gnugomez requested a review from netomi April 14, 2026 14:27
@netomi netomi merged commit 87c9fd3 into eclipse-openvsx:main Apr 15, 2026
8 checks passed
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.

Extension data is loaded multiple times when browsing the extension details page

2 participants