[ulixius9]

Describe your changes:

Hex dashboard sourceUrls were built as {hostPort}/app/projects/{id}, which isn't a route Hex serves — every "View in Hex" link from OpenMetadata was a dead link.

Real Hex project URLs are scoped by the workspace (org) UUID: {hostPort}/{workspaceId}/app/{projectId} for the published app and {hostPort}/{workspaceId}/hex/{projectId} for the notebook. No project endpoint returns the workspace id, so:

• HexApiClient.workspace_id resolves it once per run from GET /users/me (org.id), cached via cached_property.
• Published projects (lastPublishedAt set) link to /app, which viewers can open; unpublished ones link to the notebook (/hex), since there is no app to open.
• If the workspace id can't be fetched, a warning is logged and dashboards are ingested without a sourceUrl rather than with a broken one (previously a None URL would have failed the whole dashboard).

Type of change:

• Bug fix

High-level design:

N/A — small change.

Tests:

Use cases covered

• Unpublished project → notebook URL under the workspace
• Published project → app URL under the workspace
• Workspace id is fetched once per run, not per project
• /users/me failure → no URL, dashboard still ingested
• Non-default host (e.g. eu.hex.tech) with a trailing slash

Unit tests

• I added unit tests for the new/changed logic.
• Files updated: ingestion/tests/unit/topology/dashboard/test_hex.py, ingestion/tests/unit/topology/dashboard/test_hex_client.py — 46 passed

Backend integration tests

• Not applicable (no backend API changes).

Ingestion integration tests

• Not applicable — the Hex connector has no integration test suite; covered by unit tests.

Playwright (UI) tests

• Not applicable (no UI changes).

UI screen recording:

Not applicable — ingestion-only fix.

Checklist:

• I have read the CONTRIBUTING document.
• I have commented on my code, particularly in hard-to-understand areas.

🤖 Generated with Claude Code

Confidence Score: 5/5

The PR appears safe to merge; no actionable issues were found.

Summary

Hex dashboard links now include the workspace ID and open the published app or unpublished notebook.

• Hex dashboard links open the right project page.

_{Reviews (3) · Last reviewed commit: "Merge commit 'f58cfd6a26447521fd6e2b22c8..." · Reviewed by Greptile}

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[github-actions]

❌ PR checklist incomplete

This PR cannot be merged until the following are addressed on its linked issue:

• No GitHub issue is linked. Link an issue in the Development section of the PR (or add Fixes #12345 to the description). For a same-org cross-repo issue, add Fixes open-metadata/<repo>#123 to the description.

The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically.

Maintainers can bypass this check by adding the skip-pr-checks label.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[gitar-bot]

Code Review ✅ Approved

🟡 Medium risk · Fetching workspace IDs and choosing Hex app or notebook URLs changes dashboard links; unavailable or malformed identity data could omit those links.

Builds Hex dashboard source URLs scoped by workspace ID, fetched once per run from /users/me, so "View in Hex" links resolve to published /app or unpublished /hex project pages. Dashboards without a resolvable workspace ID are ingested without a sourceUrl. No issues found.

Verification

Rules No rules evaluated

Functional validation Not enabled

Set up

Auto-approval Not enabled

Set up

_{For more verification details, comment gitar display:verbose.}

Options

Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source}

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ingestion'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud