Skip to content

feat: 마이페이지 기본1 - #12

Merged
hyunhak127 merged 4 commits into
developfrom
feat/mypage
Oct 11, 2026
Merged

hyunhak127 merged 4 commits into
developfrom
feat/mypage

Conversation

@hyunhak127

Copy link
Copy Markdown
Contributor

Summary

로그인한 사용자의 마이페이지 조회와 반려견 프로필 수정 API를 구현했습니다.

Tasks

  • 마이페이지 프로필 및 산책 통계 조회 API 추가
  • 반려견 프로필 등록·수정 API 추가
  • 내 산책일지 목록·상세 조회 API 추가
  • 내가 만든 코스·좋아요한 코스 목록 및 코스 상세 조회 API 추가
  • 다른 사용자의 개인 산책 기록 조회 제한
  • 산책 제목·시간·사진 URL 저장 구조 추가
  • PostgreSQL 스키마 변경 SQL 및 API 연동 문서 추가

Verification

  • 로컬 백엔드 테스트 7개 통과
  • CI 빌드·테스트 통과 확인
  • 배포 후 Swagger API 명세 확인
  • PostgreSQL 변경 SQL 적용 및 기존 데이터 영향 확인

To Reviewer

  • 거리 응답은 미터 단위이며, 산책 날짜는 KST 기준입니다.
  • 기존 산책 기록에 시간이 없으면 durationSec는 null을 반환합니다.
  • 사진은 업로드된 URL의 저장·조회까지 지원합니다.
  • 사진 파일 업로드와 프론트 API 연결은 후속 작업입니다.
  • 사용자별 조회 범위와 스키마 변경 내용을 검토해주세요.

@hyunhak127 hyunhak127 self-assigned this Oct 8, 2026
@hyunhak127
hyunhak127 requested review from chulee-53 and hrxlou October 8, 2026 07:44

@hrxlou hrxlou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

병합 전에 수정해주세요~

WalkLogController.java 충돌 해결
기존 @operation, @securityrequirement 어노테이션은 유지하고, PR의 title, durationSec, photos 필드와 검증 및 setDetails() 저장 로직을 반영해야함.

DB 변경 SQL을 Flyway 마이그레이션으로 추가
(Flyway: DB 변경 내역을 버전별 SQL 파일로 관리하고, 배포 때 아직 적용되지 않은 파일을 자동으로 실행해 주는 도구)
src/main/resources/db/migration 안의 파일만 서버에 자동 적용되고 sql/ 폴더 파일은 자동 실행되지 않음.
기존 V1은 그대로 두고, 이번 변경은 src/main/resources/db/migration/V2__my_page.sql로 추가

추후에 DB 변경 생길때도 V3_이름.sql, V4 이런식으로 이어가면됨

@hrxlou hrxlou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

굿입니다

@hyunhak127
hyunhak127 merged commit 4c178c4 into develop Oct 11, 2026
3 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.

2 participants