Skip to content

update componentWillMount to componentDidMount#376

Open
jeffraux wants to merge 2 commits into
root-two:masterfrom
jeffraux:update-component-lifecycle
Open

update componentWillMount to componentDidMount#376
jeffraux wants to merge 2 commits into
root-two:masterfrom
jeffraux:update-component-lifecycle

Conversation

@jeffraux

Copy link
Copy Markdown

See Issue #375

@adooylabs

Copy link
Copy Markdown

@jeffraux -- I think it better to use getDerivedStateFromProps

@abhishrek

Copy link
Copy Markdown

Any plans to merge this ?
The Yellow Warnings about componentWillMount is annoying.

@Makgora

Makgora commented Jun 29, 2020

Copy link
Copy Markdown

Up

@mineshpatel1

Copy link
Copy Markdown

+1

1 similar comment
@leonimurilo

Copy link
Copy Markdown

+1

@josedaesbar

Copy link
Copy Markdown

Any news on this warning?

@zamudio-nextie

Copy link
Copy Markdown

+1

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.

8 participants