Skip to content

feat: icons for stocks and commodities#2527

Merged
onikonychev merged 2 commits intomainfrom
feat/stock-icons
Feb 27, 2026
Merged

feat: icons for stocks and commodities#2527
onikonychev merged 2 commits intomainfrom
feat/stock-icons

Conversation

@onikonychev
Copy link
Copy Markdown
Contributor

No description provided.

@onikonychev onikonychev requested a review from a team as a code owner February 20, 2026 15:29
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a number of SVG icons for various stocks and commodities. The icons appear to be sourced from TradingView. My main feedback is regarding the formatting of these new SVG files. They are currently all on a single line and lack a trailing newline. For better readability, maintainability, and to adhere to standard file conventions, I've suggested formatting them with proper indentation and adding a newline at the end. This suggestion has been made on one file but should be applied to all new SVG files in this PR.

Comment thread token-registry/img/commodities/gas.svg
@onikonychev onikonychev merged commit 03204c6 into main Feb 27, 2026
9 checks passed
@onikonychev onikonychev deleted the feat/stock-icons branch February 27, 2026 08:42
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.

3 participants