code-corps / code-corps/code-corps-api
Refactor GitHub webhook processing logic into handlers
- Vorherrschende Sprache
- Elixir
- Sterne
- 234
- Forks
- 82
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
# 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
Beitragsleitfaden
Rechercherichtung
Beginne damit, die aktuelle should_process?-Funktion, die Logik des Webhook-Ereignishandlers und den als Dispatcher beschriebenen Controller zu lesen. Verfolge, wie Repository-, Projekt- und Installations-Webhooks weitergeleitet werden. Erledigt ist die Aufgabe, wenn der Controller Ereignisse dispatcht und jeder Handler selbst entscheidet, ob sein Webhook verarbeitet werden soll.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- elixir
- Bereich
- backend-api-design
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 30/100