search

facebookarchive / fixed-data-table

A React table component designed to allow presenting thousands of rows of data.
http://facebook.github.io/fixed-data-table/
Other
4.3k stars 553 forks source link

Warning: FixedDataTable: isMounted is deprecated. #497

Open ghost opened 7 years ago

ghost commented 7 years ago

Im getting several warnings concerning isMounted deprecation?

Warning.js:36 Warning: FixedDataTable: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

Warning: FixedDataTableBufferedRows: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.
ghost commented 7 years ago

Have same Warning

acampbellb commented 7 years ago

I have been experiencing the same warnings.

emirdeliz commented 7 years ago

I have same warning

alexcroox commented 7 years ago

https://facebook.github.io/react/blog/2015/12/16/ismounted-antipattern.html

conradogarciaberrotaran commented 7 years ago

I'm having the same issue. 

Warning: FixedDataTableBufferedRows: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

Warning: FixedDataTable: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

Warning: Scrollbar: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

Is it planned fixing this issue?

jaytaylor commented 7 years ago

Also seeing this warning.

Warning: FixedDataTable: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.
alexis1990 commented 7 years ago

Solution ?

scorchiowned commented 7 years ago

Isnt this maintained anymore? Could the authors comment if we should move to perhaps FixedDataTable

RahulReddy75 commented 7 years ago

Same error.! any update on this issue?

maxdarque commented 7 years ago

Looks like it isn't maintained any more - they've got a FixedDataTable2 - see here

RahulReddy75 commented 7 years ago

@maxdarque Yes figured it. Thank you for the reply.

maxdarque commented 7 years ago

@Rahulreddy75 After trying to use multiple different react tables, I found React Table and it's been really good so far

RahulReddy75 commented 7 years ago

@maxdarque Thank you for the suggestion, will have a look. Mean while as the search was very frustrating to find a appropriate one, I ran out of patience and ended up building my own. :-D

jaytaylor commented 7 years ago

@maxdarque Thanks for the React Table tip, it looks really nice!

piyushdhamecha commented 6 years ago

having same issue...

RahulReddy75 commented 6 years ago

@piyushdhamecha Check @maxdarque's comment above... could be a good solution.