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

[feat] 약속 조회 관련 API 수정 #95

Merged
merged 7 commits into from
Aug 19, 2024
Merged

[feat] 약속 조회 관련 API 수정 #95

merged 7 commits into from
Aug 19, 2024

Conversation

chaewonni
Copy link
Member

Related issue 🛠

Work Description ✏️

  • 약속 날짜에 따라 dday에 음수와 양수가 모두 표시되도록 수정했습니다.
  • 시간 형식을 "yyyy-MM-dd HH:mm:ss"로 통일했습니다.
  • 특정 모임의 약속 목록 API에 promise가 사용자에게 속하는지 여부를 필터링하기 위한 isParticipant 쿼리 파라미터를 추가했습니다.
  • 약속 상세 정보 조회 API에 사용자가 Promise에 참여하고 있는지 여부를 나타내는 isParticipant 필드를 추가했습니다.

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

약속 상세 정보 조회 API를 수정하면서 PromiseDto를 수정하게 되었는데, 이 Dto가 약속추가 API에도 사용되어서
약속추가 API에도 "isParticipant: true" (항상 true) 를 넣도록 명세를 수정해야할 것 같은데 어떻게 생각하시나용?

add an isParticipant field to indicate whether the user is participating in the promise
Copy link
Contributor

@tkdwns414 tkdwns414 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 마지막에 질문 주신 약속 추가에서 같은 dto를 사용하는 문제는 dto를 분리하는 방향으로 가는게 맞을 것 같습니다.

저희가 아직 클린 아키텍처를 적용하기 위해 공부하는 단계이긴 하지만 dto를 함께 사용하는 것 자체가 이번처럼 사이드이펙트를 일으킬 수 있을 것 같아요~

해당 부분만 고쳐지면 될 것 같습니다!

@chaewonni chaewonni requested a review from tkdwns414 August 19, 2024 15:03
@chaewonni chaewonni merged commit 1f092ae into develop Aug 19, 2024
1 check passed
@chaewonni chaewonni deleted the feat/94 branch August 19, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 약속 조회 관련 API 수정
2 participants