[Feature/#99] 하위 목표 수정 시 하위 목표 ID를 응답으로 반환한다 #100
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1️⃣ 작업 내용 (Contents)
기존에는 클라이언트의 로컬 알림을 위해 하위 목표의 alarmDays 수정 시, 기존 alarmDays와 새로운 alarmDays의 차집합을 응답값으로 반환했습니다. 이후 차집합을 반환하지 않는걸로 요구사항이 변경되었고 하위 목표 수정 시 ID만 응답값으로 반환하도록 구현했습니다.
close : #99
작업한 내용을 정리해주세요.
2️⃣ 링크 (Links)
3️⃣ 희망 리뷰 완료 일 (Expected due date)
4️⃣ 기타 사항 (ETC)
응답으로 반환하는 하위 목표 ID의 경우, 하위 목표를 수정하기 위해 요청을 보낼때 이미 하위 목표를 식별하기 위해 함께 보내줍니다. 그렇다면 리소스를 새로 생성하는 작업이 아니고 클라이언트는 이미 하위 목표 ID를 보유하고 있기 때문에, 이 응답값이 필요한지에 대한 논의가 더 필요합니다.