Skip to content

Fix InfoOutlined export to include Icon suffix#1557

Merged
Rajesh-Nagarajan-11 merged 4 commits into
layer5io:masterfrom
rishiraj38:fix/info-outlined-icon-export
May 25, 2026
Merged

Fix InfoOutlined export to include Icon suffix#1557
Rajesh-Nagarajan-11 merged 4 commits into
layer5io:masterfrom
rishiraj38:fix/info-outlined-icon-export

Conversation

@rishiraj38
Copy link
Copy Markdown
Member

Notes for Reviewers

This PR fixes an issue with the registry page in Meshery.

Signed commits

  • Yes, I signed my commits.

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 renames the InfoOutlined export to InfoOutlinedIcon in the src/icons/InfoOutlined/index.ts file. The review feedback highlights that this is a breaking change and suggests adding an alias to maintain backward compatibility for components currently importing the icon.

Comment thread src/icons/InfoOutlined/index.ts
@rishiraj38 rishiraj38 marked this pull request as draft May 16, 2026 18:03
@rishiraj38 rishiraj38 marked this pull request as ready for review May 23, 2026 19:06
@Bhumikagarggg
Copy link
Copy Markdown
Contributor

@rishiraj38 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit 54d6cc7 into layer5io:master May 25, 2026
5 checks passed
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