Skip to content

[FIX] Enum 길이 제한 해제_(NEW: position, status)#54

Merged
Lee-Jong-Jin merged 1 commit into
mainfrom
main-temp
Nov 10, 2025
Merged

[FIX] Enum 길이 제한 해제_(NEW: position, status)#54
Lee-Jong-Jin merged 1 commit into
mainfrom
main-temp

Conversation

@Lee-Jong-Jin

@Lee-Jong-Jin Lee-Jong-Jin commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

📣 Related Issue

  • close #

📝 Summary

차장만 position 변경 안되던 문제 해결: 컬럼 길이 제한으로 인한 에러

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리스 노트

  • 리팩토링
    • 내부 데이터 매핑 설정을 최적화했습니다.

사용자 영향 사항 없음: 이번 업데이트는 내부 코드 개선 사항으로, 사용자 경험에 영향을 주지 않습니다.

@coderabbitai

coderabbitai Bot commented Nov 10, 2025

Copy link
Copy Markdown

개요

두 개의 enum 필드에서 JPA @Column 제약 조건을 제거했습니다. Position과 EmployeeStatus 필드에서 nullable = false, length = 20 제약이 삭제되었으며, 기능적 변경은 없습니다.

변경 사항

집합 / 파일 요약
Enum 필드 JPA 제약 조건 제거
src/main/java/com/sampoom/user/common/entity/BaseEmployeeEntity.java
Position과 EmployeeStatus enum 필드에서 @Column(nullable = false, length = 20) 제약 조건 제거

코드 리뷰 예상 소요 시간

🎯 1 (간단함) | ⏱️ ~3분

  • 변경 범위가 단일 파일로 제한됨
  • 제약 조건 제거만 있어 논리적 복잡성 없음
  • enum 필드 동작에 미치는 영향 확인 필요

시 🐰

열 가지 제약을 풀어주니,
자유로운 enum들이 신나게 춤을 추네!
토끼의 마법 한 줄로,
코드가 더 가벼워졌다네 🌙

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 내용의 핵심을 정확히 반영하고 있으며, 영향을 받는 필드(position, status)도 명시되어 있습니다.
Description check ✅ Passed PR 설명이 필수 섹션 대부분을 포함하고 있으며, 문제 원인과 해결 방법이 명확하게 기술되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch main-temp

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7ac5967 and 944130a.

📒 Files selected for processing (1)
  • src/main/java/com/sampoom/user/common/entity/BaseEmployeeEntity.java (0 hunks)
💤 Files with no reviewable changes (1)
  • src/main/java/com/sampoom/user/common/entity/BaseEmployeeEntity.java
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Analyze (java-kotlin)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@taemin3 taemin3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

확인했습니다

@Lee-Jong-Jin Lee-Jong-Jin merged commit c3f2791 into main Nov 10, 2025
7 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.

2 participants