Skip to content

Added missing icon exports#1345

Open
kishore08-07 wants to merge 3 commits intolayer5io:masterfrom
kishore08-07:missing-icons
Open

Added missing icon exports#1345
kishore08-07 wants to merge 3 commits intolayer5io:masterfrom
kishore08-07:missing-icons

Conversation

@kishore08-07
Copy link
Contributor

Notes for Reviewers

  • This PR adds all missing icon exports to the icons index file.
  • Added 16 missing icon exports that existed in the filesystem.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: kishore08-07 <kishorebsm8@gmail.com>
Signed-off-by: kishore08-07 <kishorebsm8@gmail.com>
Copilot AI review requested due to automatic review settings February 24, 2026 17:28
Copy link
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 adds missing icon exports to the icons index file (src/icons/index.ts) to ensure all icon components in the filesystem are properly exported. The PR also fixes a formatting issue and reorganizes exports to maintain strict alphabetical ordering.

Changes:

  • Added 16 missing icon exports that existed in the filesystem but were not exported from the index file
  • Fixed a formatting issue in the Article export (missing space between * and from)
  • Reorganized existing exports to maintain strict alphabetical order throughout the file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kishore08-07 <kishorebsm8@gmail.com>
@Rajesh-Nagarajan-11
Copy link
Member

I can see other file changes ?

Choose a reason for hiding this comment

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

@kishore08-07 this valid change

};

// export default FilterIcon;
export default ResponseIcon;
Copy link
Member

@saurabhraghuvanshii saurabhraghuvanshii Feb 24, 2026

Choose a reason for hiding this comment

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

@kishore08-07 this is not needed, because you already exporting from index.ts

}) => {
return (
<svg width={width} height={height} xmlns="http://www.w3.org/2000/svg" viewBox="0 0 40 40">
<svg width={width} height={height} xmlns="http://www.w3.org/2000/svg" viewBox="0 0 40 40" {...props}>

Choose a reason for hiding this comment

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

@kishore08-07 revert this

@kishore08-07
Copy link
Contributor Author

kishore08-07 commented Feb 24, 2026

I can see other file changes ?

@Rajesh-Nagarajan-11 Yes, those changes were made to resolve the build errors.

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.

4 participants