Separate HTTP request and HTML building portions of getEventHTML(), and refactor previewEvent() to use that instead
Los mantenedores suelen responder en 7 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
Línea de trabajo
Localiza getEventHTML() y previewEvent() en el código JavaScript y compara sus rutas de solicitud, gestión de los datos del evento y construcción de HTML. Separa la lógica compartida de representación del evento de la solicitud HTTP y, después, verifica que la representación de eventos normales y las vistas previas de formularios locales la utilicen sin duplicar la gestión de datos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
getEventHTML() currently makes an API request and modifies some values before sending it over to build the HTML. (e.g. sets the Audience label) When adding/editing an event and using the Preview function, there hasn't been a server request yet so previewEvent() has to duplicate some of the logic from getEventHTML().
Refector getEventHTML() to separate the actual request and the event data handling logic. It should be able to generate the representation of an event from an event object regardless of whether it came from an HTTP request or from local form data. Then, update previewEvent() to only make the modifications that are unique to its use case (e.g. showing print fields), but otherwise just use the same data-handling logic as the regular path.
- Lenguaje dominante
- JavaScript
- Estrellas
- 30
- Forks
- 26
- Merge medio
- 6 d 23 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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.
Más de shift-org/shift-docs
-
Unknown --db values report a TypeError instead of the intended errorPosiblemente ocupada @gangster la tomó hace 68 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
shift-org/shift-docs#1091 ·
Los mantenedores suelen responder en 7 días
-
validateRideLength accepts any Object.prototype key as a ride lengthPosiblemente ocupada @gangster la tomó hace 68 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
shift-org/shift-docs#1089 ·
Los mantenedores suelen responder en 7 días
-
Favorites page fetches every favorite because getDaily's guard is invertedPosiblemente ocupada @gangster la tomó hace 68 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
shift-org/shift-docs#1087 ·
Los mantenedores suelen responder en 7 días
-
sitemap.xml emits relative <loc> values, so search engines reject itPosiblemente ocupada @dduugg la tomó hace 94 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
shift-org/shift-docs#1082 ·
Los mantenedores suelen responder en 7 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
shift-org/shift-docs#1072 ·
Los mantenedores suelen responder en 7 días
Todos los issues de shift-org/shift-docs
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 76/100
521xueweihan/HelloGitHub#3939 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
aemsites/author-kit#19 ·
-
needs-website-changes
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día