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

코레아 1.3v 배포 #707

Merged
merged 46 commits into from
Nov 4, 2024
Merged

코레아 1.3v 배포 #707

merged 46 commits into from
Nov 4, 2024

Conversation

pp449
Copy link
Contributor

@pp449 pp449 commented Nov 4, 2024

📌 관련 이슈

  • closed: #issueNum
  • 일부 UI 개선 (모달 내 공백, 글자간격 등)
  • 상대방 리뷰에 대한 피드백을 작성해야 받은 피드백을 확인할 수 있도록 추가

✨ PR 세부 내용

jcoding-play and others added 30 commits October 24, 2024 14:05
GithubPullRequestClient라는 클래스명을 이미 사용중이라 변경
* feat: deliver 와 receiver 로 피드백을 찾는 메서드 구현

* feat: 내가 피드백을 작성했는지에 대한 isWrited 필드 추가

* feat: 받은 피드백을 보여줄 때에 내가 피드백을 작성하지 않은 상대의 피드백은 빈 응답을 내보내는 기능 구현

* test: 받은 피드백 필터링 기능 테스트 구현

* fix: deliver, receiver 를 반대로 찾던 오류 수정

* refactor: 메서드 네이밍 수정

* refactor: 피드백 반영

* refactor: BiFunction 에서 BiPredicate 로 수정

* refactor: BiFunction 에서 BiPredicate 로 재수정

* refactor: BiFunction 에서 BiPredicate 로 재재수정

* test: 다수의 assertThat 을 assertAll 로 수정

* refactor: 변수 상수화

* test: 테스트 이름 수정

* test: 테스트 이름 수정

* refactor: 변수명 직관적으로 수정

* refactor: 피드백 조회 시 roomId 도 검사하도록 수정

---------

Co-authored-by: hjk0761 <[email protected]>
* refactor: prLink 검증 로직 위치 변경

* refactor: 깃허브 pr 링크를 api url로 변환하는 기능 수정

* test: 테스트 메서드 위치 변경

* feat: PAT 발급받는 기능 구현

* refactor: 중복 코드 추상 클래스로 제거

* refactor: 클래스명 변경

GithubPullRequestClient라는 클래스명을 이미 사용중이라 변경

* refactor: pr 리뷰 정보를 가져오는 기능 메서드 분리

* refactor: 변수명 변경

* refactor: pr link 검증 로직 메서드 분리

* feat: CompletableFuture 반환하는 유틸 함수 구현

* feat: 비동기를 통해 동시 요청 보내는 기능 구현

---------

Co-authored-by: gyungchan Jo <[email protected]>
Co-authored-by: 희선이 <[email protected]>
@pp449 pp449 merged commit 3365a56 into main Nov 4, 2024
1 check 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.

5 participants