popcodeorg / popcodeorg/popcode

Preview component should not rely on prop change detection to send expressions to frame

Abierto
#1,322 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

code quality
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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.