Skip to content

Refactor: 변수명 분리 (#142) #150

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 8 commits into from
Oct 2, 2025
Merged

Refactor: 변수명 분리 (#142) #150

merged 8 commits into from
Oct 2, 2025

Conversation

@KSH0326
Copy link
Collaborator

@KSH0326 KSH0326 commented Oct 2, 2025 •
edited
Loading

📌 개요

  • 응답,요청 dto와 엔티티의 변수명을 분리

🔨 작업 내용

  • 반복 주기 변수명을 dto에서는 intervalValue를 사용,
    엔티티에서는 repeatValue사용. 프론트에는 intervalValue로 전달했는데
    내부 구현을 repeatValue로 사용하여 우선 dto와 분리 후 나중에 통합 예정

🔗 관련 이슈

Closes #142

📝 참고 사항

  • 기능 구현을 우선으로 진행하기로 결정하여 중요도가 높은 변수명 하나만 우선적으로
    수정했습니다.

✅ 체크리스트

  • 기능 동작 확인
  • 테스트 코드 작성
  • 문서/주석 추가 및 최신화

@github-actions github-actions bot changed the title Refactor: 변수명 분리 Refactor: 변수명 분리 (#142) Oct 2, 2025
@KSH0326 KSH0326 merged commit f280430 into dev Oct 2, 2025
2 checks passed
@KSH0326 KSH0326 deleted the Refactor/142 branch October 2, 2025 06:48
namgigun pushed a commit that referenced this pull request Oct 4, 2025
@KSH0326 @namgigun
Refactor: 변수명 분리 (#142) (#150)
b82957e 
* refact: 응답,요청 dto와 엔티티 변수명 분리(intervalValue)

* 주석 추가

* fix: 변수명 누락 수정

* test: test 검증 변수명 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joyewon0705 joyewon0705 joyewon0705 approved these changes

@loseminho loseminho loseminho approved these changes

@jueunk617 jueunk617 jueunk617 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@KSH0326 @joyewon0705 @loseminho @jueunk617