Skip to content

fix(ui): remove beta tag from stable services and fix Microsoft Fabric icon#27638

Open
ulixius9 wants to merge 1 commit intomainfrom
remove_beta
Open

fix(ui): remove beta tag from stable services and fix Microsoft Fabric icon#27638
ulixius9 wants to merge 1 commit intomainfrom
remove_beta

Conversation

@ulixius9
Copy link
Copy Markdown
Member

@ulixius9 ulixius9 commented Apr 22, 2026

Summary

  • Remove the Beta tag from services that are now stable: OpenLineage, Wherescape, Cassandra, AlationSink, Cockroach, OpenSearch, ThoughtSpot, Ranger, Grafana, and Timescale.
  • Fix Microsoft Fabric and Microsoft Fabric Pipeline icons on the service creation screen. getServiceIcon normalizes the key with toLowerCase(), but the icon map used camelCase keys (microsoftFabric, microsoftFabricPipeline), so the lookup missed and the tile fell back to the default database icon. Lowercased the keys to match.

Test plan

  • Open service creation → Database services; confirm the Microsoft Fabric tile shows the Microsoft Fabric logo (not the generic DB cylinder).
  • Open service creation → Pipeline services; confirm the Microsoft Fabric Pipeline tile shows the logo.
  • Confirm the following no longer display a Beta badge: OpenLineage, Wherescape, Cassandra, AlationSink, Cockroach, OpenSearch, ThoughtSpot, Ranger, Grafana, Timescale.
  • Confirm services still expected to be Beta remain marked: SSIS, SSAS, Epic, Hex, ServiceNow, Dremio, Collibra, Mulesoft, Microsoft Fabric, Microsoft Fabric Pipeline, BurstIQ, StarRocks, SFTP, GoogleDrive, Informix, MicrosoftAccess.

🤖 Generated with Claude Code

…c icon

- Drop OpenLineage, Wherescape, Cassandra, AlationSink, Cockroach, OpenSearch,
  ThoughtSpot, Ranger, Grafana, and Timescale from BETA_SERVICES — these are
  stable and should no longer show the Beta badge.
- Fix Microsoft Fabric / Microsoft Fabric Pipeline icons: getServiceIcon
  normalizes keys via toLowerCase(), but the map used camelCase keys
  (microsoftFabric, microsoftFabricPipeline), so lookup missed and the tiles
  fell back to the default database icon. Lowercased the keys to match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 22, 2026 17:02
@ulixius9 ulixius9 requested a review from a team as a code owner April 22, 2026 17:02
@github-actions github-actions Bot added Ingestion safe to test Add this label to run secure Github workflows on PRs labels Apr 22, 2026
@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented Apr 22, 2026

Code Review ✅ Approved

Removes the beta tag from stable services and corrects the Microsoft Fabric icon reference. No issues found.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the UI service creation experience by (1) removing “Beta” badges for connectors that are now considered stable and (2) fixing Microsoft Fabric icon resolution by aligning icon-map keys with getServiceIcon’s normalization behavior.

Changes:

  • Remove multiple stable connector types from the BETA_SERVICES list so they no longer render the Beta badge.
  • Fix Microsoft Fabric and Microsoft Fabric Pipeline service icons by lowercasing the icon-map keys to match getServiceIcon(iconKey.toLowerCase()) normalization.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
openmetadata-ui/src/main/resources/ui/src/utils/ServiceIconUtils.ts Lowercase Microsoft Fabric icon-map keys so normalized lookups resolve to the correct Fabric icon instead of the default DB icon.
openmetadata-ui/src/main/resources/ui/src/constants/ServiceType.constant.ts Remove now-stable services from BETA_SERVICES to stop showing Beta badges for those connectors.

@github-actions
Copy link
Copy Markdown
Contributor

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 61%
61.99% (60426/97469) 42.02% (31688/75403) 45.04% (9512/21117)

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

🟡 Playwright Results — all passed (12 flaky)

✅ 3699 passed · ❌ 0 failed · 🟡 12 flaky · ⏭️ 89 skipped

Shard Passed Failed Flaky Skipped
✅ Shard 1 481 0 0 4
🟡 Shard 2 654 0 2 7
🟡 Shard 3 664 0 2 1
🟡 Shard 4 646 0 2 27
✅ Shard 5 611 0 0 42
🟡 Shard 6 643 0 6 8
🟡 12 flaky test(s) (passed on retry)
  • Features/BulkEditEntity.spec.ts › Glossary (shard 2, 1 retry)
  • Features/Glossary/GlossaryWorkflow.spec.ts › should display correct status badge color and icon (shard 2, 1 retry)
  • Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
  • Features/Table.spec.ts › Tags term should be consistent for search (shard 3, 1 retry)
  • Pages/Customproperties-part2.spec.ts › entityReferenceList shows item count, scrollable list, no expand toggle (shard 4, 1 retry)
  • Pages/DataContracts.spec.ts › Semantic with Contains Operator should work for Tier, Tag and Glossary (shard 4, 1 retry)
  • Features/AutoPilot.spec.ts › Create Service and check the AutoPilot status (shard 6, 1 retry)
  • Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
  • Pages/Lineage/LineageRightPanel.spec.ts › Verify custom properties tab IS visible for supported type: searchIndex (shard 6, 1 retry)
  • Pages/ODCSImportExport.spec.ts › Multi-object ODCS contract - object selector shows all schema objects (shard 6, 1 retry)
  • Pages/UserDetails.spec.ts › Admin user can edit teams from the user profile (shard 6, 1 retry)
  • Pages/Users.spec.ts › Permissions for table details page for Data Consumer (shard 6, 1 retry)

📦 Download artifacts

How to debug locally
# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ingestion safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants