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

[Feature/#99] 하위 목표 수정 시 하위 목표 ID를 응답으로 반환한다 #100

Merged
merged 2 commits into from
Oct 21, 2023

Conversation

jemlog
Copy link
Collaborator

@jemlog jemlog commented Oct 21, 2023

1️⃣ 작업 내용 (Contents)

기존에는 클라이언트의 로컬 알림을 위해 하위 목표의 alarmDays 수정 시, 기존 alarmDays와 새로운 alarmDays의 차집합을 응답값으로 반환했습니다. 이후 차집합을 반환하지 않는걸로 요구사항이 변경되었고 하위 목표 수정 시 ID만 응답값으로 반환하도록 구현했습니다.

close : #99

작업한 내용을 정리해주세요.

2️⃣ 링크 (Links)

3️⃣ 희망 리뷰 완료 일 (Expected due date)

4️⃣ 기타 사항 (ETC)

응답으로 반환하는 하위 목표 ID의 경우, 하위 목표를 수정하기 위해 요청을 보낼때 이미 하위 목표를 식별하기 위해 함께 보내줍니다. 그렇다면 리소스를 새로 생성하는 작업이 아니고 클라이언트는 이미 하위 목표 ID를 보유하고 있기 때문에, 이 응답값이 필요한지에 대한 논의가 더 필요합니다.

- 기존에 수정된 요일의 차집합을 반환하던 응답을 하위 목표 ID만 반환하도록 수정했습니다.
@jemlog jemlog added the feature New feature or request label Oct 21, 2023
@jemlog jemlog self-assigned this Oct 21, 2023
@jemlog jemlog changed the title [Feature/#99] 하위 목표 수정 시 하위 목표 ID를 응답으로 반환한다. [Feature/#99] 하위 목표 수정 시 하위 목표 ID를 응답으로 반환한다 Oct 21, 2023
@jemlog jemlog merged commit 06edd1e into dev Oct 21, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant