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.

Sign up for GitHub

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

Jump to bottom

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

Closed
marquesm91 opened this issue Jun 7, 2023 · 3 comments
Closed

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

marquesm91 opened this issue Jun 7, 2023 · 3 comments
Labels
Merged in next Merged but not live

Comments

@marquesm91
Copy link

marquesm91 commented Jun 7, 2023 •
edited

Do you want to request a feature or report a bug?
feature

What is the current behavior?
When I am trying to use react-toastify with react@18.2.0 I am getting this warning. The React MR to print this warning on console is here.

I am pretty sure this is a small refactor to remove this warning and I could work on that 😄

Captura de Tela 2023-06-07 às 14 36 20

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

What is the expected behavior?
No warning should appear

Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?

React 18.2
@fkhadra
Copy link
Owner

fkhadra commented Jun 10, 2023

Hey this is addressed in v10, I'm currently working on it

@fkhadra fkhadra mentioned this issue Jun 11, 2023
Merged
@fkhadra fkhadra added the Merged in next Merged but not live label Jun 16, 2023
@maiveasna24
Copy link

got this issues in next-js version 14

@fkhadra fkhadra closed this as completed Jan 14, 2024
@rwb196884
Copy link

Still broken in 10.0.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Merged in next Merged but not live
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fkhadra @marquesm91 @rwb196884 @maiveasna24