Skip to content

feat(marketplace): add Go to Details button to info dialog#494

Open
larsgeorge-db wants to merge 1 commit into
mainfrom
feat-marketplace-go-to-details
Open

feat(marketplace): add Go to Details button to info dialog#494
larsgeorge-db wants to merge 1 commit into
mainfrom
feat-marketplace-go-to-details

Conversation

@larsgeorge-db
Copy link
Copy Markdown
Collaborator

Summary

  • Add an optional onGoToDetails callback prop to EntityInfoDialog and render a "Go to Details" button in the footer, using the same ExternalLink icon as the marketplace tile's quick-link button.
  • Footer now renders whenever either onSubscribe or onGoToDetails is provided (instead of only for subscribe).
  • Wire onGoToDetails on the marketplace product info modal (marketplace-view.tsx) so users can jump from the preview straight to /my-products/{id} — mirroring the tile's icon button.

Test plan

  • Open the marketplace, click any product tile to open the info modal.
  • Verify a "Go to Details" button appears in the footer between "Close" and "Subscribe" with the ExternalLink icon.
  • Click it and confirm the modal closes and the app navigates to the product detail page.
  • Verify the tile's existing ExternalLink icon button still works.
  • Verify other usages of EntityInfoDialog (asset explorer, data products/contracts/domains views, discovery section, metadata panel) still render correctly with no footer change since they don't pass onGoToDetails.

Adds an optional onGoToDetails callback to EntityInfoDialog that renders
a "Go to Details" button (with the ExternalLink icon used on tiles) in
the dialog footer. Wired up on the marketplace product modal so users
can jump from the info preview to the full product detail page,
mirroring the icon-button on the product tile.
@larsgeorge-db larsgeorge-db requested a review from a team as a code owner June 3, 2026 16:19
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