Skip to content

Address Barchart icon#1452

Merged
saurabhraghuvanshii merged 3 commits intolayer5io:masterfrom
indevi:barchart
Apr 29, 2026
Merged

Address Barchart icon#1452
saurabhraghuvanshii merged 3 commits intolayer5io:masterfrom
indevi:barchart

Conversation

@indevi
Copy link
Copy Markdown
Contributor

@indevi indevi commented Apr 29, 2026

Notes for Reviewers

This PR fixes #1446

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>
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 introduces a new BarchartIcon component and integrates it into the icons library. The review identified a module resolution error in the Barchart index file and a copy-paste error in the data-testid attribute of the SVG element.

Comment thread src/icons/Barchart/index.ts Outdated
Comment thread src/icons/Barchart/BarchartIcon.tsx Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: shivam tyagi <tyagishivam.dev@gmail.com>
Copy link
Copy Markdown
Contributor

@kishore08-07 kishore08-07 left a comment

Choose a reason for hiding this comment

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

@saurabhraghuvanshii saurabhraghuvanshii merged commit d9e653c into layer5io:master Apr 29, 2026
5 checks passed
@indevi indevi deleted the barchart branch April 29, 2026 12:36
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.

[Feature]Add BarChart icon

3 participants