Skip to content

Add OracleDB and SAP font icons - #2455

Closed
gayaldassanayake wants to merge 1 commit into
wso2:mainfrom
gayaldassanayake:agent/add-oracle-sap-brand-icons
Closed

Add OracleDB and SAP font icons#2455
gayaldassanayake wants to merge 1 commit into
wso2:mainfrom
gayaldassanayake:agent/add-oracle-sap-brand-icons

Conversation

@gayaldassanayake

Copy link
Copy Markdown
Contributor

Summary

  • add monochrome OracleDB and SAP JCo SVG glyphs to the shared WSO2 VS Code icon font
  • expose the existing bi-oracledb and bi-sap icon names to UI-toolkit consumers

Root cause

Ballerina artifact cards resolve brand registry entries through the shared icon font. Product Integrator already had matching side-panel SVG assets, but the shared font did not contain these two glyphs, so the artifact cards rendered blank icons.

Impact

Oracle CDC and SAP JCo can use the same brand keys in both artifact cards and the Product Integrator project explorer.

Validation

  • regenerated font metadata and verified bi-oracledb and bi-sap are present
  • rebuilt the Ballerina VS Code extension and verified both glyphs are included in the packaged VSIX

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 84845b57-a2cb-4c56-8a13-23bd726d2bb5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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