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

이슈 상세보기 및 댓글 조회 기능 구현 #38

Merged
merged 32 commits into from
Sep 8, 2024
Merged

Conversation

lvalentine6
Copy link
Member

@lvalentine6 lvalentine6 commented Sep 3, 2024

🚀 개요

  • 이슈 상세보기와 해당 이슈에 달린 댓글 조회 기능을 추가하였습니다.
  • 이슈 url을 실제 깃허브 url과 동일하게 하기 위해 issue_Id 대신에 issue_number로 수정하였습니다.
  • 메서드명과 변수명을 조금 더 명확하게 수정하였습니다.
  • 이슈 상세보기에 대한 테스트를 추가하였습니다.
  • 아래는 응답 예시입니다.
{
    "issueDetailDto": {
        "number": 45,
        "title": "이슈가 왔어요",
        "state": "open",
        "user": {
            "login": "dokkisan",
            "avatar_url": "https://avatars.githubusercontent.com/u/117690393?v=4"
        },
        "created_at": "2024-07-28T16:25:11",
        "updated_at": "2024-09-01T19:18:46",
        "body": "신선한 이슈입니다",
        "labels": [
            {
                "id": 7074218588,
                "name": "good first issue",
                "color": "7057ff"
            },
            {
                "id": 7074218591,
                "name": "help wanted",
                "color": "008672"
            }
        ]
    },
    "comments": [
        {
            "id": 2323466532,
            "user": {
                "login": "lvalentine6",
                "avatar_url": "https://avatars.githubusercontent.com/u/77956808?v=4"
            },
            "created_at": "2024-09-01T19:18:46",
            "updated_at": "2024-09-01T19:18:46",
            "body": "정말로 신선한가요?"
        }
    ]
}

Issue-ID: issuefy-##

📗 유형

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🔎 상세 내용

✅ Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.

@lvalentine6 lvalentine6 requested a review from dokkisan September 3, 2024 22:01
@lvalentine6 lvalentine6 self-assigned this Sep 3, 2024
@lvalentine6 lvalentine6 added bug Something isn't working enhancement New feature or request labels Sep 4, 2024
Copy link
Contributor

@dokkisan dokkisan left a comment

Choose a reason for hiding this comment

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

GitHub API를 이용한 로직은 뚝딱 만들어 내시는군요! ㅎㅎ
너무 잘 작성되어서 리뷰 남길 게 별로 없었는데요..!
이번 작업도 고생 많으셨습니다 👍👍

Copy link

sonarqubecloud bot commented Sep 8, 2024

Copy link
Contributor

@dokkisan dokkisan left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 머지하겠습니다! 😃

@dokkisan dokkisan merged commit d3c3380 into main Sep 8, 2024
2 checks passed
@lvalentine6
Copy link
Member Author

🎉 This PR is included in version 1.18.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants