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

Use ES6 default parameters instead of deprecated defaultProps property #95

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

Conversation

fnogatz
Copy link

@fnogatz fnogatz commented Jul 10, 2024

As mentioned in #86 and #64, the current version of the icons results in warnings like the following:

Warning: UilBrushAlt: Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead.

In this PR, we adjust the build/generate.js script which is called on npm run generate, and newly create the icon components.


The work on this PR was sponsored by SUSI&James GmbH.

@htoann
Copy link

htoann commented Jul 22, 2024

Please merge as soon as possiable

@willibix
Copy link

@tarunmangukiya and @rankarpan please review and merge this. Same update to Iconscout/react-unicons-solid would be aprciated.

@fnogatz
Copy link
Author

fnogatz commented Aug 12, 2024

Fair point, @willibix. I'll happily add the same update to https://github.com/Iconscout/react-unicons-solid as well once this change here gets approved and merged (ping @rankarpan).

@dkildar
Copy link

dkildar commented Aug 21, 2024

I see approvement, but PR still open. Any updates? @htoann @fnogatz

@htoann
Copy link

htoann commented Aug 21, 2024

We need a people who has permission to merge it

@Tooni
Copy link

Tooni commented Aug 30, 2024

i have made another package that implements the same icons with TypeScript and default params

https://github.com/Tooni/iconscout-unicons-react

@dkildar
Copy link

dkildar commented Sep 3, 2024

@Tooni amazing! thanks

@fnogatz
Copy link
Author

fnogatz commented Sep 23, 2024

*ping @rankarpan

@htoann
Copy link

htoann commented Nov 4, 2024

Any update?

@fnogatz
Copy link
Author

fnogatz commented Nov 26, 2024

Hey @imzedi, any chance this PR gets reviewed and merged?

@imzedi
Copy link
Contributor

imzedi commented Dec 17, 2024

Yes @fnogatz, Will check them out soon

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.

7 participants