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

Refactor GitHub webhook processing logic into handlers

Open
#1,258 0 comments 0 reactions 0 assignees View on GitHub
needs clarification Priority: Low
Dominant language
Elixir
Stars
234
Forks
82
PR merge metrics
No merged PRs in 30d

Description

# 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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.