Skip to content

[FIX]: Serializable을 BaseEntity로 이동 #73

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 6, 2025
Merged

[FIX]: Serializable을 BaseEntity로 이동 #73

merged 1 commit into from
Oct 6, 2025

Conversation

@Gooraeng
Copy link
Collaborator

@Gooraeng Gooraeng commented Oct 4, 2025 •
edited
Loading

제목(필수): [TYPE]: 제목 예) [FEAT]: 회원가입 기능 추가

제목 규칙 자세히 보기
  • 형식: [TYPE]: 제목
  • 제한: 50자 이내, 첫 글자 대문자, 명령문
  • TYPE: FEAT FIX REFACTOR COMMENT STYLE TEST CHORE INIT

무엇을 / 왜

  • 무엇(What)
    • Serializable 구현체를 BaseEntity로 이동
  • 왜(Why)
    • 프로덕션 환경에서 일반 로그인 시 Id 가 null 로 받아와지던 문제가 생겨, 로그인 절차가 제대로 처리되지 않았습니다.
    • 해당 부분을 BaseEntity로 조정하고 로컬에서 redis를 이용한 로그인 테스트 결과 Id 값도 잘 받아와지는 것 확인했습니다.

어떻게(요약) — 3줄 이내

  • User 쪽 직렬화 구현을 BaseEntity로 이동

영향 범위

  • API 변경
  • DB 마이그레이션
  • Breaking Change
  • 보안/권한 영향
  • 문서/가이드 업데이트 필요

체크리스트

  • 타입 라벨 부착 (FEAT/FIX/REFACTOR/COMMENT/STYLE/TEST/CHORE/INIT)
  • 로컬/CI 테스트 통과
  • 영향도 점검 완료
  • 주석/문서 반영(필요 시)

ToDo (선택)

  • 할 일 1
  • 할 일 2

스크린샷/증빙(선택)

이슈 연결 (자동)

Closes #74

@lcs9317 lcs9317 mentioned this pull request Oct 4, 2025
Closed
11 tasks
@Gooraeng Gooraeng added the FIX 버그 수정 label Oct 4, 2025
johnbosco0414
johnbosco0414 approved these changes Oct 4, 2025
View reviewed changes
Copy link
Collaborator

@johnbosco0414 johnbosco0414 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

shihan00321
shihan00321 approved these changes Oct 5, 2025
View reviewed changes
Copy link
Collaborator

@shihan00321 shihan00321 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@Gooraeng Gooraeng merged commit 9915f46 into main Oct 6, 2025
3 checks passed
@Gooraeng Gooraeng deleted the fix/move-serializable-to-baseentity branch October 6, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shihan00321 shihan00321 shihan00321 approved these changes

@johnbosco0414 johnbosco0414 johnbosco0414 approved these changes

Assignees

@Gooraeng Gooraeng

Labels

FIX 버그 수정

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FIX]: Serializable을 BaseEntity로 이동

4 participants

@Gooraeng @shihan00321 @johnbosco0414