Skip to content

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Jan 20, 2026

🔍 개요


🚀 주요 변경 내용

  • council 테이블의 instagram_url 컬럼을 instagram_user_name 으로 수정했습니다.

  • 그에 따라 인스타 아이디 정책에 맞는 유효성 제약을 수정했습니다.

  • 아이디에 @는 포함하지 않는 가정하에 작성했습니다.


💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

Copy link
Contributor

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

이 PR은 council 테이블의 Instagram 관련 필드를 URL에서 사용자명(username)으로 변경하는 작업입니다. 데이터베이스 스키마, 엔티티 모델, DTO, 서비스 레이어를 포함한 전체 애플리케이션 계층에서 일관되게 변경이 이루어졌습니다.

Changes:

  • 데이터베이스 컬럼명을 instagram_url에서 instagram_user_name으로 변경하고 기존 URL 데이터를 사용자명으로 변환하는 마이그레이션 추가
  • Instagram 사용자명 형식에 맞는 유효성 검증 규칙으로 업데이트 (영문/숫자/점/밑줄만 허용, 최대 30자)
  • 엔티티, DTO, 서비스 레이어에서 필드명과 메서드 파라미터를 instagramUrl에서 instagramUserName으로 일관되게 변경

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/main/resources/db/migration/V12__alter_council_instagram_column.sql Instagram URL 컬럼을 username 컬럼으로 변경하고, 기존 URL 데이터를 파싱하여 사용자명만 추출하는 마이그레이션 스크립트
src/main/java/gg/agit/konect/domain/council/model/Council.java 엔티티 필드명을 instagramUrl에서 instagramUserName으로 변경 및 관련 메서드 업데이트
src/main/java/gg/agit/konect/domain/council/dto/CouncilCreateRequest.java 필드명 변경 및 Instagram 사용자명 형식에 맞는 유효성 검증 규칙 추가
src/main/java/gg/agit/konect/domain/council/dto/CouncilUpdateRequest.java 필드명 변경 및 Instagram 사용자명 형식에 맞는 유효성 검증 규칙 추가
src/main/java/gg/agit/konect/domain/council/dto/CouncilResponse.java Response DTO 필드명 변경 및 매핑 로직 업데이트
src/main/java/gg/agit/konect/domain/council/service/CouncilService.java 서비스 메서드에서 파라미터명 변경

@dh2906 dh2906 merged commit ba84303 into main Jan 20, 2026
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.

2 participants