Skip to content

leetcode-study 외 저장소에 Week 경고 댓글이 달리는 버그 수정#25

Merged
sounmind merged 2 commits intomainfrom
fix/filter-non-allowed-repos
Apr 16, 2026
Merged

leetcode-study 외 저장소에 Week 경고 댓글이 달리는 버그 수정#25
sounmind merged 2 commits intomainfrom
fix/filter-non-allowed-repos

Conversation

@DaleSeo
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo commented Apr 11, 2026

Closes #24

GitHub App이 leetcode-study 외 저장소에 Week 경고 댓글을 작성하는 버그를 수정합니다. projects_v2_item webhook에는 payload.repository 필드가 없어 상위 저장소 필터를 우회하고 있었고, /check-weeks 엔드포인트에는 repo_name 검증이 빠져 있었습니다. 두 핸들러 모두 ALLOWED_REPO 상수로 검증하도록 수정하고, 회귀 테스트와 테스트 자동화를 위한 CI 워크플로우를 추가했습니다.

@DaleSeo DaleSeo self-assigned this Apr 11, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 591ce76 Commit Preview URL

Branch Preview URL
Apr 11 2026, 10:04 PM

@DaleSeo DaleSeo marked this pull request as ready for review April 11, 2026 22:14
@DaleSeo DaleSeo requested a review from a team April 12, 2026 11:36
@sounmind
Copy link
Copy Markdown
Member

sounmind commented Apr 16, 2026

버그 픽스 건이라 일단 빠르게 병합하겠습니다.

@sounmind sounmind merged commit 4913987 into main Apr 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

달레 UI 프로젝트의 PR에 리트코드 스터디 주차 설정 관련 코멘트가 달림

2 participants