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] Inquiry API replied flag 추가 #113

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

shj1081
Copy link
Contributor

@shj1081 shj1081 commented Nov 13, 2024

작성자: @shj1081

#112

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Reviewers/Assignees/Labels을 알맞게 설정했나요?

작업 내역

  • InquiryDetailResponse (DTO) 에 replied Boolean flag 추가
  • 문의 단건 조회, 문의 업데이트, 문의 생성 의 경우 답변 확인 여부 logic 추가
  • test code 수정
  • inquiry.html 추가
  • rebuild 후 affected html 업데이트 (index, project, inquiry)

비고

  • 일단 Issue 에 올라온대로 단건 조회의 경우에만 replied flag를 추가하였습니다
  • 만약 list 조회 시에도 필요 시 추가하도록 하겠습니다
  • 과제 제안의 경우에도 동일 issue로 올라와 있어 close는 하지 않도록 하겠습니다

@shj1081 shj1081 added 🐞 Bug 버그 수정 🎯 API API 개발 관련 Inquiry Domain labels Nov 13, 2024
@shj1081 shj1081 self-assigned this Nov 13, 2024
Copy link
Contributor

@yesjuhee yesjuhee left a comment

Choose a reason for hiding this comment

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

static/docs의 index.html도 변경내용 commit 되어야할 것 같은데 생략된 것 같습니당

@shj1081
Copy link
Contributor Author

shj1081 commented Nov 13, 2024

static/docs의 index.html도 변경내용 commit 되어야할 것 같은데 생략된 것 같습니당

index.html, project.html, inquiry.html rebuild 해서 commit 하도록 하겠습니다

반영 commit: e3354aa

Copy link
Contributor

@yesjuhee yesjuhee left a comment

Choose a reason for hiding this comment

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

👍
LGTM입니당 감사합니다!

@shj1081 shj1081 merged commit 6ae301a into develop Nov 14, 2024
1 check passed
@shj1081 shj1081 deleted the fix/112-include-inquiry-replied-flag branch November 14, 2024 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 API API 개발 관련 🐞 Bug 버그 수정 Inquiry Domain
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants