-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FEAT] 행복루틴 테마 리스트 조회 #48
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨어요! 병합 전 API 테스트 꼭 부탁드려요 :)
src/main/java/com/soptie/server/routine/controller/HappinessRoutineController.java
Outdated
Show resolved
Hide resolved
@Service | ||
@Transactional(readOnly = true) | ||
@RequiredArgsConstructor | ||
public class HappinessRoutineServiceImpl implements HappinessRoutineService{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 스타일의 통일을 위해서 Service와 { 사이에 한 칸 띄어쓰기 부탁드려요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헛 감사합니다
✨ Related Issue
📝 기능 구현 명세
🐥 추가적인 언급 사항
사실 아직 테스트 제대로 못해봄 일단 개발해보고 내일 시큐리티랑 같이 테스트하고 코드 수정하겠음 ㅎㅎ;;