Skip to content

Extension Placement + Tooltip Lazy Fetch#2142

Merged
shangyian merged 2 commits into
DataJunction:mainfrom
shangyian:ext-tooltip-fixes
May 17, 2026
Merged

Extension Placement + Tooltip Lazy Fetch#2142
shangyian merged 2 commits into
DataJunction:mainfrom
shangyian:ext-tooltip-fixes

Conversation

@shangyian
Copy link
Copy Markdown
Collaborator

@shangyian shangyian commented May 17, 2026

Summary

  • Modify extension placement to be between Settings and Custom Metadata for the node editor page
  • Fix tooltip lazy-fetch so that it accepts a fetchNodeDetails callback and progressively enhances the popover with details when they arrive

Test Plan

  • PR has an associated issue: #
  • make check passes
  • make test shows 100% unit test coverage

Deployment Plan

shangyian added 2 commits May 17, 2026 08:09
… and progressively enhances the popover with details when they arrive
@netlify
Copy link
Copy Markdown

netlify Bot commented May 17, 2026

Deploy Preview for thriving-cassata-78ae72 canceled.

Name Link
🔨 Latest commit 05a9c5c
🔍 Latest deploy log https://app.netlify.com/projects/thriving-cassata-78ae72/deploys/6a09da633197b9000814dbbd

@shangyian shangyian changed the title Ext tooltip fixes Extension Placement + Tooltip Lazy Fetch May 17, 2026
@shangyian shangyian marked this pull request as ready for review May 17, 2026 15:11
@shangyian shangyian merged commit d943fab into DataJunction:main May 17, 2026
21 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.

1 participant