popcodeorg / popcodeorg/popcode
Preview component should not rely on prop change detection to send expressions to frame
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 <Preview> component looks for changes in the list of console expressions to determine what if any expressions to send to the frame for evaluation. Component behavior that depends on both previous and current state is considerably harder to reason about and should be avoided. Instead, console entries should have state indicating whether they have been sent to a frame for evaluation; this should be updated synchronously after the frame initially posts the expression. The <Preview> component should simply consume all entries that are not marked as evaluation started.
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
Empieza siguiendo el componente y el console-entry state que se utiliza cuando el frame publica una expresión por primera vez. Identifica dónde se marcan las entradas después de que comienza la evaluación y, a continuación, actualiza el flujo para que Preview consuma las entradas que no estén marcadas como iniciadas. Se considera terminado cuando la entrega de la expresión ya no depende de comparar los valores de props anteriores y actuales.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- frontend
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100