Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces the SettingsRemoteIcon component and adds it to the icons index. The review feedback suggests switching from a default export to a named export to maintain consistency with other icons in the codebase, while also addressing minor formatting issues such as an unnecessary leading empty line and a missing trailing newline.
Signed-off-by: Prarthana <226818177+prarii@users.noreply.github.com>
prarii
force-pushed
the
feature/prarii/1420
branch
from
98c350e to
e0fc843
Compare
kishore08-07
left a comment
kishore08-07
left a comment
There was a problem hiding this comment.
@prarii
Take this https://github.com/layer5io/sistent/pull/1427/changes as reference template and refactor the changes accordingly.
|
hi @kishore08-07, sorry to bother but please could you recheck the template link? you've linked my own pull request. |
@prarii Sorry about that, my mistake. Here’s the correct link: https://github.com/layer5io/sistent/pull/1324/changes. |
2 participants
Notes for Reviewers
This PR fixes #1420
Signed commits