Skip to content

feature: 회원 도메인 엔티티와 저장소 추가 - #9

Merged
ddingmin merged 1 commit into
mainfrom
feature/user-entities
Jul 27, 2026
Merged

feature: 회원 도메인 엔티티와 저장소 추가#9
ddingmin merged 1 commit into
mainfrom
feature/user-entities

Conversation

@ddingmin

Copy link
Copy Markdown
Collaborator

왜 바꿨나요?

회원 인증 기능의 기반이 되는 회원, 로그인 계정, 리프레시 토큰을 데이터베이스에 저장하고 도메인 규칙과 함께 관리하기 위해 추가했습니다.

무엇을 바꿨나요?

  • user 기능 아래에 JPA 엔티티와 JpaRepository를 추가했습니다.
  • 모든 엔티티에 생성·수정 시각을 기록하고, 공개 setter 없이 도메인 함수로 상태를 변경하도록 구성했습니다.
  • user, group, photo 기능별 패키지 구조와 도메인 작성 지침을 추가했습니다.
  • 도메인 규칙, JPA 영속화, Repository CRUD를 검증하는 테스트를 추가했습니다.

확인

  • ./gradlew check 통과

@github-actions

Copy link
Copy Markdown

🧪 테스트 결과

성공 실패 스킵
26 0 0

상세 테스트 결과 보기

@ddingmin
ddingmin marked this pull request as ready for review July 27, 2026 22:36
@ddingmin
ddingmin requested a review from swonny as a code owner July 27, 2026 22:36

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1274681c74

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

import jakarta.persistence.Table

@Entity
@Table(name = "\"user\"")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 운영 스키마에 새 엔티티 테이블을 추가하세요

devprod 프로필은 ddl-auto: validate를 사용하지만, 저장소 전체를 확인해도 user, login_account, refresh_token 테이블을 만드는 SQL 또는 마이그레이션 도구 설정이 없습니다. 따라서 이전 커밋의 데이터베이스로 이 버전을 시작하면 Hibernate 스키마 검증에서 필수 테이블이 없다는 오류가 발생해 애플리케이션이 기동되지 않으므로, 엔티티와 함께 운영 스키마 마이그레이션을 제공해야 합니다.

AGENTS.md reference: AGENTS.md:L88-L88

Useful? React with 👍 / 👎.

@ddingmin
ddingmin merged commit 3a2356c into main Jul 27, 2026
2 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.

1 participant