Fixed bug where extensions() query was executed multiple times - #6447
Conversation
…ach navigation item) instead once
🦋 Changeset detectedLatest commit: 0a0665f 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 |
Fixed bug where extensions() query was executed multiple times
There was a problem hiding this comment.
Pull request overview
This PR removes per-menu-item extension lookup/opening logic from the sidebar’s SingleItem component, addressing the reported issue where the extensions() query was being triggered multiple times due to repeated extension hook usage.
Changes:
- Removed
useExtensions(...)usage fromSingleItemto avoid per-item extension fetching. - Removed
getMenuItemExtension(...)-based click handling and now relies onmenuItem.onClickwhen present.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6447 +/- ##
========================================
Coverage 43.58% 43.58%
========================================
Files 2585 2585
Lines 44991 44985 -6
Branches 10148 10541 +393
========================================
- Hits 19609 19607 -2
+ Misses 25342 25338 -4
Partials 40 40 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.