comunes/todos-contra-el-fuego-web
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

cleanup: silence our own React dev-console warnings

Browse source 
UNSAFE_ rename of our 4 componentWillReceiveProps (FireStats, Fires, FromNow,
SelectionMap) and Authenticated/Public component propType func -> elementType.
Rest of the console noise is legacy react-* dev-mode warnings, absent in prod.
This commit is contained in:
vjrj 2026-07-18 05:47:44 +02:00
parent 9feaca9d0b
commit a997aa7cae
6 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

4
imports/ui/components/Authenticated/Authenticated.js
View file

@ -17,7 +17,9 @@ const Authenticated = ({ loggingIn, authenticated, component, path, exact, ...re
Authenticated.propTypes = {
loggingIn: PropTypes.bool.isRequired,
authenticated: PropTypes.bool.isRequired,
component: PropTypes.func.isRequired,
// elementType (not func): route components may be HOC-wrapped (translate(),
// memo, forwardRef), which are objects, not plain functions.
component: PropTypes.elementType.isRequired,
};
export default Authenticated;

2
imports/ui/components/FromNow/FromNow.js
View file

@ -18,7 +18,7 @@ class FromNow extends Component {
};
}
componentWillReceiveProps(nextProps) {
UNSAFE_componentWillReceiveProps(nextProps) {
if (this.props.when !== nextProps.when) {
// console.log(`Next when ${nextProps.when}`);
this.setState({

2
imports/ui/components/Public/Public.js
View file

@ -21,7 +21,7 @@ const Public = ({
Public.propTypes = {
loggingIn: PropTypes.bool.isRequired,
authenticated: PropTypes.bool.isRequired,
component: PropTypes.func.isRequired
component: PropTypes.elementType.isRequired
};
export default Public;

2
imports/ui/components/SelectionMap/SelectionMap.js
View file

@ -60,7 +60,7 @@ class SelectionMap extends Component {
}
}
componentWillReceiveProps(nextProps) {
UNSAFE_componentWillReceiveProps(nextProps) {
const nextCenter = nextProps.center[0] ? nextProps.center : this.state.center;
const nextMarker = nextProps.center[0] ? nextProps.center : this.state.marker;
this.setState({

2
imports/ui/pages/Fires/Fires.js
View file

@ -37,7 +37,7 @@ class Fire extends React.Component {
};
}
componentWillReceiveProps(nextProps) {
UNSAFE_componentWillReceiveProps(nextProps) {
if (this.props.when !== nextProps.when || this.props.loading !== nextProps.loading || this.props.notfound !== nextProps.notfound) {
// console.log(`Next when ${nextProps.when}`);
if (nextProps.fire && (nextProps.alert || nextProps.active || nextProps.fromHash)) {

2
imports/ui/pages/FiresMap/FireStats.js
View file

@ -17,7 +17,7 @@ class FireStats extends Component {
};
}
componentWillReceiveProps(nextProps) {
UNSAFE_componentWillReceiveProps(nextProps) {
if (this.props.lastCheck !== nextProps.lastCheck ||
this.props.lastFireDetected !== nextProps.lastFireDetected ||
this.props.loadingAll !== nextProps.loadingAll