code-corps / code-corps/code-corps-api

Refactor GitHub webhook processing logic into handlers

Offen
#1,258 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
needs clarification Priority: Low
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.