Skip to content

[backport core/1.34] Fix desktop menu docs links regression#7501

Merged
christian-byrne merged 1 commit intocore/1.34from
backport-7181-to-core-1.34
Dec 15, 2025
Merged

[backport core/1.34] Fix desktop menu docs links regression#7501
christian-byrne merged 1 commit intocore/1.34from
backport-7181-to-core-1.34

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Dec 15, 2025

Backport of #7181 to core/1.34

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

## Summary
- make `useExternalLink` rely on the global i18n locale so it can be
used safely outside setup
- restore `electronAdapter` to use the shared `useExternalLink` helper
for docs URLs and static links

## Motivation
Desktop menu items disappeared because a top-level call to
`useExternalLink` in `electronAdapter` triggered `useI18n` at
module-eval time, throwing and blocking extension registration. By
making the composable global-locale-only and using it in
`electronAdapter`, the module can load without setup context while
preserving link behavior.

## Testing
- pnpm typecheck
- pnpm lint:fix

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-7181-Fix-desktop-menu-docs-links-regression-2c06d73d36508157ae48cff078b9173e)
by [Unito](https://www.unito.io)
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner December 15, 2025 18:21
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Dec 15, 2025
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 15, 2025
@github-actions
Copy link

github-actions bot commented Dec 15, 2025

🎨 Storybook Build Status

Build completed successfully!

⏰ Completed at: 12/15/2025, 06:23:08 PM UTC

🔗 Links


🎉 Your Storybook is ready for review!

@github-actions
Copy link

github-actions bot commented Dec 15, 2025

🎭 Playwright Test Results

⚠️ Tests passed with flaky tests

⏰ Completed at: 12/15/2025, 07:24:56 PM UTC

📈 Summary

  • Total Tests: 485
  • Passed: 463 ✅
  • Failed: 0
  • Flaky: 12 ⚠️
  • Skipped: 10 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 456 / ❌ 0 / ⚠️ 12 / ⏭️ 10
  • chromium-2x: Deployment failed
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 6 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

@christian-byrne christian-byrne merged commit a7d3650 into core/1.34 Dec 15, 2025
32 of 33 checks passed
@christian-byrne christian-byrne deleted the backport-7181-to-core-1.34 branch December 15, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants