rawgraphs / rawgraphs-app

A web interface to create custom vector-based visualizations on top of RAWGraphs core
https://rawgraphs.io
Apache License 2.0
8.66k stars 1.84k forks source link

"findDOMNode is deprecated" on data type selection #233

Closed iosonosempreio closed 8 months ago

iosonosempreio commented 4 years ago

Once data is loaded on the data grid, the first time you try to change a data value (eg. from number to date) the app triggers this warning.

Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of Transition which is inside StrictMode. Instead, add a ref directly to the element you want to reference. Learn more about using refs safely here: https://fb.me/react-strict-mode-find-node
    in div (at DataGrid.js:99)
    in Transition (created by Fade)
    in Fade (created by Overlay)
    in Overlay (created by Overlay)
    in Overlay (at DataGrid.js:80)
    in DataTypeSelector (at DataGrid.js:142)
    in div (at DataGrid.js:141)
    in HeaderRenderer (created by HeaderCell)
    in span (created by SortableHeaderCell)
    in span (created by SortableHeaderCell)
    in SortableHeaderCell (created by HeaderCell)
    in div (created by HeaderCell)
    in HeaderCell (created by HeaderRow)
    in div (created by HeaderRow)
    in HeaderRow (created by ForwardRef(DataGrid))
    in div (created by ForwardRef(DataGrid))
    in ForwardRef(DataGrid) (at DataGrid.js:224)
    in DataGrid (at DataLoader.js:268)
    in div (created by Col)
    in Col (at DataLoader.js:390)
    in div (created by Row)
    in Row (at DataLoader.js:389)
    in div (created by Col)
    in Col (at DataLoader.js:388)
    in div (created by Row)
    in Row (at DataLoader.js:310)
    in DataLoader (at App.js:43)
    in div (created by Col)
    in Col (at Section.js:9)
    in div (created by Row)
    in Row (at Section.js:8)
    in div (created by Container)
    in Container (at Section.js:7)
    in Section (at App.js:42)
    in div (at App.js:40)
    in App (at src/index.js:9)
    in StrictMode (at src/index.js:8)
bianchimro commented 4 years ago

This is an issue with react-boostrap, that uses a deprecated react api. Should be fixed in a future update of the lib and it should not be a problem in production.