RedHatInsights / insights-chrome

Chroming for Insights apps
MIT License
27 stars 130 forks source link

add brain icon for AI/ML allservices section #2840

Closed InsaneZein closed 3 months ago

InsaneZein commented 3 months ago

for RHCLOUD-32731 Adds the correct icon for AI/ML

codecov-commenter commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 63.61%. Comparing base (539a1f0) to head (08f07f3).

:exclamation: Current head 08f07f3 differs from pull request most recent head 1f7cded

Please upload reports for the commit 1f7cded to get more accurate results.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/RedHatInsights/insights-chrome/pull/2840/graphs/tree.svg?width=650&height=150&src=pr&token=GuRwyW1uUf&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RedHatInsights)](https://app.codecov.io/gh/RedHatInsights/insights-chrome/pull/2840?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RedHatInsights) ```diff @@ Coverage Diff @@ ## master #2840 +/- ## ======================================= Coverage 63.60% 63.61% ======================================= Files 201 201 Lines 4317 4318 +1 Branches 778 778 ======================================= + Hits 2746 2747 +1 Misses 1562 1562 Partials 9 9 ``` | [Files](https://app.codecov.io/gh/RedHatInsights/insights-chrome/pull/2840?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RedHatInsights) | Coverage Δ | | |---|---|---| | [src/components/FavoriteServices/ServiceIcon.tsx](https://app.codecov.io/gh/RedHatInsights/insights-chrome/pull/2840?src=pr&el=tree&filepath=src%2Fcomponents%2FFavoriteServices%2FServiceIcon.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RedHatInsights#diff-c3JjL2NvbXBvbmVudHMvRmF2b3JpdGVTZXJ2aWNlcy9TZXJ2aWNlSWNvbi50c3g=) | `100.00% <100.00%> (ø)` | |
epwinchell commented 3 months ago

@InsaneZein No need to add an SVG, we have this in Patternfly http://patternfly-react.surge.sh/icons/ (BrainIcon)

InsaneZein commented 3 months ago

@epwinchell fixed