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

[FIX] 병합 충돌 대응 #82

Merged

Conversation

thguss
Copy link
Member

@thguss thguss commented Jan 12, 2024

✨ Related Issue

📝 기능 구현 명세

회원 별 행복루틴 조회를 구현하기 전에, 병합 충돌을 피하기 위해서 임의로 해당 브랜치를 병합했습니다.
병합 중 이미 존재하는 에러가 있어 원활한 진행을 위해 임의로 수정해두었습니다,,

🐥 추가적인 언급 사항

보고 괜찮으면 바로 승인하고 머지해주세요!

@thguss thguss added sohyeon 소현 작업 fix labels Jan 12, 2024
@thguss thguss requested a review from csb9427 January 12, 2024 04:25
@thguss thguss self-assigned this Jan 12, 2024
Copy link
Contributor

@csb9427 csb9427 left a comment

Choose a reason for hiding this comment

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

항상 원할하게 프로젝트를 돌아가게 해주셔서 감사합니다

Comment on lines -48 to +50
private HappinessRoutine findRoutine(Long id) {
return happinessRoutineRepository.findById(id)
private HappinessSubRoutine findRoutine(Long id) {
return happinessSubRoutineRepository.findById(id)
Copy link
Contributor

Choose a reason for hiding this comment

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

수정 감사합니다

Copy link
Contributor

@Chan531 Chan531 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@thguss thguss merged commit b234430 into feature/#58-happiness-routine-post-api Jan 12, 2024
@thguss thguss deleted the fix/#81-routine-post-api branch January 12, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix sohyeon 소현 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants