Skip to content
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.

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

#339 엔티티 ID strategy AUTO -> IDENTITY 및 데이터 변경으로 인한 실행오류 수정 #340

Merged
merged 3 commits into from
Jul 29, 2024

Conversation

psyeon1120
Copy link
Member

📍 이슈 번호


🛠️ 작업 내용

  1. 규칙적으로 1개씩 ID가 증가되도록 AUTO에서 IDENTITY로 변경했습니다.
  2. 값을 반환하지 않는 API는 원래 이렇게 null을 넣어서 호출했습니다.
    public static <T> ResponseCustom<T> success() { return new ResponseCustom<>(null); }
    근데 ReturnCode를 도입하면서 null을 넣으면 ReturnCode로 인식해서 오류가 떴어요. 그래서 아무런 값도 넣지않는 생성자 추가했습니다.

🎸 기타 사항

  • 냉장고쪽에서 지금사용하지 않는 코드때문에 오류나서 지웠습니다.

@psyeon1120 psyeon1120 added 🔨fix 버그 해결 💿 entity 엔티티의 수정 labels Jul 29, 2024
@psyeon1120 psyeon1120 self-assigned this Jul 29, 2024
Comment on lines +22 to +30
// 성공
public ResponseCustom() {
this.data = null;
this.timeStamp = LocalDateTime.now();
this.httpStatus = HttpStatus.OK.value();
this.message = ReturnCode.SUCCESS.getMessage();
this.code = ReturnCode.SUCCESS.getCode();
}

Copy link
Member

Choose a reason for hiding this comment

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

좋습니도 🙌 꼼꼼하게 봐줘서 고마옹

@sojungpp sojungpp merged commit f2e57c8 into develop Jul 29, 2024
1 check passed
@sojungpp sojungpp deleted the entity/#339-id-strategy branch July 29, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💿 entity 엔티티의 수정 🔨fix 버그 해결
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[entity] 엔티티 ID strategy AUTO -> IDENTITY 변경
2 participants