Skip to content

Conversation

@wlgns12370
Copy link
Member

No description provided.

wodydl0 and others added 30 commits February 24, 2025 22:28
* Update main-cd.yml (#17)

* Update main-cd.yml/v3
* RINGUS-40 feat: 북마크 생성 및 삭제 기능 구현

* RINGUS-40 chore: 멘티 스키마 변경에 따른 flywayV2 생성
* RINGUS-45 feat: SessionMemberId 어노테이션 기능 구현

* RINGUS-45 refactor: 자기 정보 조회시 멤버 타입 반환 추가
* RINGUS-44 feat: 증명서 및 이미지 로직 관심사 분리

* RINGUS-44 feat: 프로필 이미지 조회 api 및 에러코드 작성

* RINGUS-44 fix: 조회 기능 삭제
* RINGUS-48 refactor: 멘토 엔티티 변경

* RINGUS-48 chore: 멘토 엔티티 변경에 따른 flyway 스키마 변경
* RINGUS-49 docs: 내 프로필 상세 조회 Docs 작성
* RINGUS-52 feat: 내 멤버 조회 imgUrl 필드 추가

* RINGUS-52 fix: 멘토 멘티 request DTO에서 받는 Id를 SessionId로 받도록 변경

* RINGUS-52 fix: jakarta.persistence:jakarta.persistence-api:3.0.0 버전 충돌 에러 해결
* RINGUS-46 feat: 멘토링 신청 엔티티 및 에러코드 작성

* RINGUS-46 fix: domain 명 apply -> mentoring 변경

* RINGUS-46 feat: 멘토링 제안 api 작성

* RINGUS-46 feat: docs 추가 및 파라미터 오류 수정
* RINGUS-58 feat: 멘토링 멘토, 멘티 연관관계 매핑

* RINGUS-58 feat: 멘토링 테이블 flyway 추가

* RINGUS-58 chore: multipart 최대 파일 사이즈 변경
* RINGUS-27 feat:  멘토 프로필 목록 조회 API 개발

* RINGUS-27 feat: 멘토 프로필 목록 조회 API 구현
* Update main-cd.yml (#17)

* Fix/main cd/v3 (#18)

* Update main-cd.yml (#17)

* Update main-cd.yml/v3

* RINGUS-27 refactor: 커서 기반 페이지네이션 로그 삭제

* RINGUS-61 feat: 내 프로필 조회 기능 구현

---------

Co-authored-by: Jaeyeong Park <[email protected]>
* RINGUS-62 feat: 북마크 멘토 목록 조회 기능 추가

* RINGUS-62 fix: 로그인 아닐 때 멘토 목록 조회 안되는 오류 수정
* RINGUS-64 refactor: 닉네임 유효 검사 POST로 변경

* RINGUS-64 feat: 멘토기준 멘토링 신청 받은 멘티 목록 조회

* RINGUS-64 test: 멘토링 테스트 sql 작성
* RINGUS-64 refactor: 닉네임 유효 검사 POST로 변경

* RINGUS-64 feat: 멘토기준 멘토링 신청 받은 멘티 목록 조회

* RINGUS-64 test: 멘토링 테스트 sql 작성

* RINGUS-64 fix: git 스테이징 안된 파일push
* RINGUS-64 refactor: 닉네임 유효 검사 POST로 변경

* RINGUS-64 feat: 멘토기준 멘토링 신청 받은 멘티 목록 조회

* RINGUS-64 test: 멘토링 테스트 sql 작성

* RINGUS-64 fix: git 스테이징 안된 파일push

* RINGUS-64 refactor: 닉네임 중복 검사 POST에서 GET으로 변경
wlgns12370 and others added 9 commits March 24, 2025 16:02
* RINGUS-66 rename: commissioned필드 suggested로 변경

* RINGUS-66 feat: 멘토 멘토링 수락 기능 구현

* RINGUS-66 docs: 멘토 멘토링 수락 docs 작성
* RINGUS-66 rename: commissioned필드 suggested로 변경

* RINGUS-66 feat: 멘토 멘토링 수락 기능 구현

* RINGUS-66 docs: 멘토 멘토링 수락 docs 작성

* RINGUS-66 fix: HTTP PUT 메서드 추가
* RINGUS-70 rename: Image 필드명 변경

* RINGUS-70 rename: 기존의 ImgUrl 필드 image로 변경
* RINGUS-70 rename: Image 필드명 변경

* RINGUS-70 rename: 기존의 ImgUrl 필드 image로 변경

* RINGUS-70 rename: 기존의 ImgUrl 필드 image로 변경

* RINGUS-70 feat: 멘토링 count 쿼리 구현

* RINGUS-70 feat: 북마크 확인 쿼리 구현
* RINGUS-70 rename: Image 필드명 변경

* RINGUS-70 rename: 기존의 ImgUrl 필드 image로 변경

* RINGUS-70 rename: 기존의 ImgUrl 필드 image로 변경

* RINGUS-70 feat: 멘토링 count 쿼리 구현

* RINGUS-70 feat: 북마크 확인 쿼리 구현

* RINGUS-70 fix: 쿼리Dsl 생성자 주입 추가
# Conflicts:
#	src/main/java/es/princip/ringus/application/member/service/MemberService.java
#	src/main/java/es/princip/ringus/application/mentor/service/MentorService.java
#	src/main/java/es/princip/ringus/application/mentor/service/MyMentorService.java
#	src/main/java/es/princip/ringus/domain/bookmark/BookmarkRepository.java
#	src/main/java/es/princip/ringus/domain/mentor/MentorQueryDslRepositoryImpl.java
#	src/main/java/es/princip/ringus/domain/mentoring/MentoringRepository.java
#	src/main/java/es/princip/ringus/presentation/member/dto/MentorProfileResponse.java
#	src/main/java/es/princip/ringus/presentation/mentor/MentorController.java
#	src/main/java/es/princip/ringus/presentation/mentor/dto/MentorCardResponse.java
#	src/main/java/es/princip/ringus/presentation/mentor/dto/MentorDetailResponse.java
#	src/main/java/es/princip/ringus/presentation/mentor/dto/MyMentorResponse.java
@wlgns12370 wlgns12370 requested a review from Copilot March 29, 2025 04:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the data types for mentoring count fields across several DTOs and modifies factory methods to accept a dynamic mentoring count, while also adding query support for mentoring and bookmark features to enhance the mentor-related endpoints.

  • Changed mentoringCount type from int to Long in multiple DTOs.
  • Updated static factory methods to accept mentoringCount as an argument.
  • Integrated QueryDSL support for mentoring counting and bookmark checking.

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/main/java/es/princip/ringus/presentation/mentor/dto/MyMentorResponse.java Changed mentoringCount type and updated factory method signature.
src/main/java/es/princip/ringus/presentation/mentor/dto/MentorDetailResponse.java Changed mentoringCount type and updated factory method signature.
src/main/java/es/princip/ringus/presentation/mentor/dto/MentorCardResponse.java Changed mentoringCount type and added a bookmarked field in factory methods.
src/main/java/es/princip/ringus/presentation/member/dto/MentorProfileResponse.java Changed mentoringCount type and updated factory method signature.
src/main/java/es/princip/ringus/domain/mentoring/MentoringRepository.java Added extension of MentoringQueryDslRepository interface.
src/main/java/es/princip/ringus/domain/mentoring/MentoringQueryDslRepositoryImpl.java Added custom QueryDSL method for mentoring count.
src/main/java/es/princip/ringus/domain/bookmark/BookmarkRepository.java Extended repository to implement custom QueryDSL interface.
src/main/java/es/princip/ringus/domain/bookmark/BookmarkQueryDslRepositoryImpl.java Added custom QueryDSL method to check bookmarked status.
src/main/java/es/princip/ringus/application/mentor/service/MyMentorService.java Injected mentoringRepository to obtain mentoring count.
src/main/java/es/princip/ringus/application/mentor/service/MentorService.java Updated method calls to include mentoring count from mentoringRepository.
src/main/java/es/princip/ringus/application/member/service/MemberService.java Injected mentoringRepository and updated profile construction with mentoring count.
.github/workflows/main-cd.yml Added environment file creation and fixed Gradle permission settings.
Comments suppressed due to low confidence (1)

.github/workflows/main-cd.yml:46

  • [nitpick] The workflow script appears to set Gradle permissions twice; consider consolidating these steps to avoid redundant commands.
sudo chown -R $USER:$USER ~/.gradle

Comment on lines 15 to 16
.from(mentoring)
.where(mentoring.mentor.id.eq(mentorId).and(mentoring.mentee.memberId.eq(memberId)))
Copy link

Copilot AI Mar 29, 2025

Choose a reason for hiding this comment

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

Using the 'mentoring' entity in the Bookmark repository implementation appears to be incorrect. This query should reference the Bookmark entity instead to ensure the correct data is fetched.

Suggested change
.from(mentoring)
.where(mentoring.mentor.id.eq(mentorId).and(mentoring.mentee.memberId.eq(memberId)))
.from(bookmark)
.where(bookmark.mentor.id.eq(mentorId).and(bookmark.mentee.memberId.eq(memberId)))

Copilot uses AI. Check for mistakes.
wodydl0 and others added 19 commits March 31, 2025 16:02
* RINGUS-65 fix :이미지 파일 로직 fileMember에 추가하도록 변경

* RINGUS-65 feat: S3 증명서 접근 권한 분리 및 presigned url 적용

* RINGUS-65 feat: fileMember 조회 api 개발(관리자)

* RINGUS-65 feat: api Docs 설정
* RINGUS-65 feat: S3 증명서 접근 권한 분리 및 presigned url 적용

* RINGUS-65 feat: api Docs 설정

* RINGUS-77 feat: TimeTraceAop 설정
* RINGUS-74 fix: bookmark 여부 확인 쿼리Dsl 수정

* RINGUS-74 fix: 기본 목록 조회 response 오버로딩 기능 추가

* RINGUS-74 feat: 한국어 번역을 위한 Enum 매핑

* RINGUS-74 fix: mentor-cursor 누락된 프로필 정보 삽입
* RINGUS-74 fix: bookmark 여부 확인 쿼리Dsl 수정

* RINGUS-74 fix: 기본 목록 조회 response 오버로딩 기능 추가

* RINGUS-74 feat: 한국어 번역을 위한 Enum 매핑

* RINGUS-74 fix: mentor-cursor 누락된 프로필 정보 삽입

* RINGUS-74 feat: Category 한국어 번역 기능 구현

* RINGUS-74 feat: MentoringField, MentoringTopic 한국어 번역 기능 구현

* RINGUS-74 feat: Days 한국어 번역 기능 구현

* RINGUS-74 refactor: Copilot, memberId null 예외 처리 로직 추가

* RINGUS-74 refactor: enum 변경 데이터 test sql에 반영
* RINGUS-65 feat: S3 증명서 접근 권한 분리 및 presigned url 적용

* RINGUS-65 feat: api Docs 설정

* RINGUS-78 feat: 멘토링 신청 취소 및 거절

* RINGUS-78 feat: mentoring status 스키마 수정

* RINGUS-78 fix: CreateMentoringRequest의 토픽 클래스 수정
* RINGUS-74 fix: mentor category 한국어 테이블 수정

* RINGUS-74 fix: MentorDetail Email 필드 추가

* RINGUS-74 fix: MenteeDetail Email 필드 추가
* RINGUS-83 feat: 로그인 여부와 멘토, 멘티 여부 확인 기능 추가

* RINGUS-83 refactor: 주석 제거

---------

Co-authored-by: jbh010204 <j@2180130>
* RINGUS-87 feat: 포트폴리오 업로드
* feat: S3 파일 업로드 수정

* feat: flyway schema 추가
* RINGUS-74 fix: mentor category 한국어 테이블 수정

* RINGUS-74 fix: MentorDetail Email 필드 추가

* RINGUS-74 fix: MenteeDetail Email 필드 추가

* RINGUS-74 fix: MentorDetail email 필드 제거 및 MyMentorDetail email 필드 추가
* RINGUS-74 fix: mentor category 한국어 테이블 수정

* RINGUS-74 fix: MentorDetail Email 필드 추가

* RINGUS-74 fix: MenteeDetail Email 필드 추가

* RINGUS-74 fix: MentorDetail email 필드 제거 및 MyMentorDetail email 필드 추가

* RINGUS-74 fix: Portfolio 빌더 패턴 적용 및 MyMentorDetail image 필드 추가
* RINGUS-74 feat: Notification Domain 구현

* RINGUS-84 feat: 멘토링 신청 알림 기능 구현
* RINGUS-67 refactor: 예외 처리 code도 반환하게 변경

* RINGUS-67 refactor: password 검증 패턴 유틸로 변경

* RINGUS-67 feat: 비밀번호 재설성 기능 구현

---------

Co-authored-by: jbh010204 <j@2180130>
Co-authored-by: jbh010204 <j@2180130>
* RINGUS-92 fix: Hotfix

* RINGUS-92 refactor: 비밀번호 재설정 api 세션 검증 필요 없음으로 변경

---------

Co-authored-by: jbh010204 <j@2180130>
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.

4 participants