Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #195 several icons that were missing #204

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ganiserb
Copy link

This a new PR like this one #196 but right, since I tested it this time.

I've found that some icons were not defined, or incorrectly classified.

I am also not sure if the .icondb.js file shoud even be in the repo. This is recreated on every build, correct?

@phorozan
Copy link

Any update on this?

@ganiserb
Copy link
Author

@phorozan Looks like the author is not maintaining this repo anymore, but in the mean time you can use this directly from this fix branch. That's what we are doing curently.

package.json line

"@exuanbo/file-icons-js": "git+https://github.com/DivvyHQ/file-icons-js.git#fix-for-some-icons-build",

@exuanbo Can you please take a look at this?

@ganiserb ganiserb changed the title Fixed several icons that were missing Fixes #195 several icons that were missing Aug 23, 2022
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.

None yet

2 participants