popcodeorg / popcodeorg/popcode
Refactor instructions editor to follow controlled component pattern
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 191
- Forks
- 143
- Merge medio
- 4 d 11 h
- PR fusionados (30 d)
- 5
Descripción
Currently the instructions editor is an uncontrolled component, which is to say that intermediate in-progress editing state is stored only in the DOM.
React officially recommends using controlled components instead, which stores intermediate state in React (in our case Redux) state, and has the form value mirror the props given. This is probably the right approach here (but maybe it’s not! Further research may be called for).
Probably best not to sweat this too much until #1394 is done (or at least concurrently with that task).
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 revisando el editor de instrucciones y la documentación enlazada de React sobre componentes controlados; después, comprueba cómo afecta #1394 al trabajo. La refactorización estará terminada cuando el estado intermedio de edición se almacene en el estado de React o Redux y el valor del formulario refleje sus props.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, react, redux
- Área
- frontend
- Tipo de issue
- Refactorización
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100