Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

adds another operations icon #168

Merged
merged 5 commits into from
Apr 20, 2020
Merged

Conversation

caydie-tran
Copy link
Contributor

adding this icon
image

@caydie-tran caydie-tran added the minor Increment the minor version when merged label Apr 17, 2020
@justinanastos
Copy link
Contributor

This adds an operations icon named src/icons/svgs/navigation/icon-operations.svg when we already have src/icons/svgs/navigation/icon-operations-sl.svg. I never planned on having an icon with the same name as -sl; so this causes the library to get confused because when it converts svg files to TypeScript files it uses the filename to make the new filename and strips off -sl. So icon-operations.svg and icon-operations-sl.svg will both be read and will output the same filename, causing an overwrite. Then storybook will look at the source .svg files because it needs to know if an svg is a streamline icon or not, which is why we see the same icon on there twice.

TLDR; Delete src/icons/svgs/navigation/icon-operations-sl.svg in this PR and you'll be all set.

I should add a test that verifies that there aren't duplicates so that we know exactly why things are behaving strangely strange.

@caydie-tran caydie-tran merged commit a216734 into master Apr 20, 2020
@apollo-bot2
Copy link
Collaborator

🚀 PR was released in v4.1.0 🚀

@apollo-bot2 apollo-bot2 added the released This issue/pull request has been released. label Apr 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants