Refactor GitHub webhook processing logic into handlers

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

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
30/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
elixir
Área
backend-api-design

Línea de trabajo

Empieza leyendo la función should_process? actual, la lógica del manejador de eventos de webhook y el controller descrito como el dispatcher. Rastrea cómo se enrutan los webhooks de repository, project e installation. Se considera terminado cuando el controller despacha los eventos y cada handler se encarga de decidir si debe procesarse su webhook.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

needs clarification Priority: Low

Problem

There are "classes" of webhooks, some of which pertain to the repository and project and other that don't (installation, e.g.). As CC adds more webhook handlers for various reasons, the should_process? function will become increasingly further away from the handler logic, so this should eventually move down into the event handler logic, with the controller merely being a dispatcher.

https://github.com/code-corps/code-corps-api/pull/1254#issuecomment-347049874

Lenguaje dominante
Elixir
Estrellas
234
Forks
82
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

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.

Más de code-corps/code-corps-api

Todos los issues de code-corps/code-corps-api

Issues similares

Más issues de Elixir

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.