Skip to content

Conversation

@Soundbar91
Copy link
Collaborator

🔍 개요


🚀 주요 변경 내용

어드민 히스토리 로깅 어노테이션 추가

  • 기존에는 API PATH를 통해 도메인 이름, 엔티티 ID 결정
  • URL의 규칙이 불규칙하여 분기 로직이 증가
  • 이를 해결하기 위해 커스텀 어노테이션 추가
    • 도메인 이름과 엔티티 ID를 설정
    • AOP에서 이를 추출하여 DB에 저장
  • 해당 커밋에서 확인 가능 : e7b5db6, ff108f0

DomainType 리펙토링

  • Enum 네이밍이 맞지 않는 상황
  • 복수형 + 언더스코어 반영 및 flyway 추가

💬 참고 사항

  • 어드민 패키지 하위에 모두 적용했는데, 커밋별로 보시면 됩니다.
    • 개인정보가 포함된 API, GET API는 안 붙입니다.

✅ Checklist (완료 조건)

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

@github-actions github-actions bot added Team User 유저 팀에서 작업할 이슈입니다 리팩터링 리팩터링을 위한 이슈입니다 labels Nov 2, 2025
@github-actions
Copy link

github-actions bot commented Nov 2, 2025

Unit Test Results

672 tests   671 ✔️  1m 15s ⏱️
165 suites      1 💤
165 files        0

Results for commit c8753ea.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@BaeJinho4028 BaeJinho4028 left a comment

Choose a reason for hiding this comment

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

매번 고생 많으십니다.
이전 레거시 기능이지만, 굳이 메서드마다 어노테이션을 붙이지 않더라도 로깅할 수 있지 않을까 싶습니다. 클래스 단위로도 도메인 구분이 될거고, 클래스 단위가 아니더라도 전체적으로 로깅할 수 있지 않을까 싶습니다.
그럼에도 현재 기능에서 좋은 방향으로 개선된 것 같습니다. 👍

Copy link
Member

@asa9874 asa9874 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다👍

@Soundbar91 Soundbar91 merged commit e975900 into develop Nov 8, 2025
5 checks passed
@Soundbar91 Soundbar91 deleted the refactor/2053-admin-history branch November 8, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team User 유저 팀에서 작업할 이슈입니다 리팩터링 리팩터링을 위한 이슈입니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[유저] 어드민 히스토리 리펙토링

4 participants