Refactor GitHub webhook processing logic into handlers
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- elixir
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu bằng cách đọc hàm should_process? hiện tại, logic của trình xử lý sự kiện webhook và controller được mô tả là dispatcher. Theo dõi cách các webhook repository, project và installation được định tuyến. Hoàn thành khi controller dispatch các sự kiện và mỗi handler tự chịu trách nhiệm quyết định webhook của mình có nên được xử lý hay không.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Elixir
- Star
- 234
- Fork
- 82
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của code-corps/code-corps-api
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
code-corps/code-corps-api#1385 ·
-
Cannot Create new GitHub App Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
code-corps/code-corps-api#1384 ·
-
Track anon Users that register Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
code-corps/code-corps-api#1383 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
code-corps/code-corps-api#1382 · 2 bình luận ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 bình luận · 1 người được giao ·
Tất cả issue của code-corps/code-corps-api
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
agentjido/jido_harness#80 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
sevenseacat/cinder#235 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
phoenixframework/phoenix#6847 ·