Tutorial page "Lifting State Up" demo can give incorrect boiling verdict
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 11.8k
- Forks
- 7.9k
- Merge medio
- 1 d 11 h
- PR fusionados (30 d)
- 11
Descripción
Because the Boiling verdict uses the temperature in celsius , if the user puts in fahrenheit 211.9999 this will incorrectly report that it will boil. This is because the conversion to celsius is limited to 3 digits and gets rounded up to 100.

A fix for this is to change BoilingVerdict to accept scale and temperature and the code to do a slightly different check for each:
function BoilingVerdict(props) {
if (props.scale === 'c' ? props.temperature >= 100 : props.temperature >= 212) {
return <p>The water would boil.</p>;
}
return <p>The water would not boil.</p>;
}
<BoilingVerdict
scale={this.state.scale}
temperature={this.state.temperature}
/>

I know that I could do a fork and pull request but this is about as far as my interest in this subject goes so anyone else is free to fix this admittedly tiny issue.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza por la página del tutorial “Lifting State Up” y localiza el componente BoilingVerdict y su uso. Comprueba el resultado para valores Fahrenheit justo por debajo de 212, después actualiza el componente y confirma que 211.9999 no indica ebullición, mientras que los valores umbral sí lo siguen haciendo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, react
- Área
- documentation
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100