Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat : 깃허브 이슈 구현 & 이슈리스트 페이징처리 구현 #47

Merged
merged 11 commits into from
Sep 17, 2024

Conversation

Mouon
Copy link
Member

@Mouon Mouon commented Aug 30, 2024

요약 (Summary)

  • 깃허브 이슈 구현 작업 입니다.

🔑 변경 사항 (Key Changes)

  • WebhookController : 깃허브 웹훅에 등록할 /webhook/studyroomId/{studyroomId} 엔드포인트를 만들었습니다. 해당 엔드포인트를 깃허브 웹훅에 등록하면 해당주소로 요청을 보내며, 저희 측 서버는 그 이슈를 스터디룸과 매핑해서 저장하고 STOMP로 해당 내용을 보냅니다. 구독주소 : /topic/issues/{studyroomId}
    • getWebhookUrl : 웹훅 주소를 발급받는 API입니다.
  • GithubIssueService : getGithubIssueList 깃허브 이슈 리스트를 가져오는 서비스 계층 코드입니다.
  • GithubIssueDSL : 쿼리DSL을 통해 페이징을 구현하였습니다. 이전 페이지의 마지막 요소아이디와 한번에 가져올 데이터의 개수를 받습니다.)
  • GithubIssueController : getIssueList 깃허브 이슈 리스트를 가져오는 API입니다. ( 커서 기반 페이징 처리를 하였습니다. )

📝 리뷰 요구사항 (To Reviewers)

  • 웹훅 특성상 배포후 작업을 할수 밖에 없었습니다... 배포된 서버에서 웹훅주소를 발급받고 해당 웹훅을 등록하여 정상적으로 웹훅이 저장되는지 확인해주세요!
  • STOMP가 정상적으로 발송되는지 봐주세요! [테스트 페이지 첨부해 두었습니다!]
  • 리스트 요청이 페이징 처리되어 잘 반환되는지 확인해주세요! 명세서

확인 방법

배포환경 테스트라 쿼리문은 필요없을거같아 제공하지않겠습니다!

명세서 <- 페이징에대한 추가설명이 있습니다.

  • https://www.linkode.site/webhookUrl?studyroomId=3 여기로 GET 요청을 보내주세요!
  • https://lahezy.tistory.com/111 이 블로그를 참고해서 웹훅 주소를 등록하고 이슈를 발생시켜주세요 (푸쉬, 피알)
  • DB에서 저장이 제대로 되었나 확인해주세요!
  • https://www.linkode.site/studyroom/issue?studyroomId=3 이 요청이 리스트를 잘 반환하는지 확인해주세요!

웹훅 주소 발급 [GET] <웹훅 특성상 배포하면서 개발 하였습니다..>

https://www.linkode.site/webhookUrl?studyroomId=3

깃허브 이슈 리스트 [GET] <리스트는 로컬에만 구현되어있습니다!>

https://localhost:8080/studyroom/issue?studyroomId=2&limit=2&lastGithubIssueId=2

성공 예시

스크린샷 2024-08-30 오후 6 51 22 스크린샷 2024-08-30 오후 7 06 03

@Mouon Mouon added the Feature Make Feat label Sep 6, 2024
@Mouon Mouon requested a review from jsilver01 September 6, 2024 05:36
@Mouon Mouon changed the title Feat : 깃허브 이슈 구현 Feat : 깃허브 이슈 구현 & 이슈리스트 페이징처리 구현 Sep 10, 2024
Copy link
Member

@jsilver01 jsilver01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다~~ 너무 수고하셨습니당 머지할게용

@jsilver01 jsilver01 merged commit 53778b8 into develop Sep 17, 2024
@jsilver01 jsilver01 deleted the LINKODE-19-BE-깃허브-이슈 branch September 17, 2024 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Make Feat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants