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] Inquiry API / controller test / RestDocs 구현 #75

Open
wants to merge 20 commits into
base: feat/45-inquiry-api/base
Choose a base branch
from

Conversation

shj1081
Copy link
Contributor

@shj1081 shj1081 commented Aug 23, 2024

작성자: @shj1081

#45

체크 리스트

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

작업 내역

Inquiry API 구현 / Controller test / RestDocs

  • createProjectInquiry
  • getInquiry / getInquiries
  • updateInquiry
  • deleteInquiry
  • createInquiryReply
  • getInquiryReply
  • updateInquiryReply
  • deleteInquiryReply

Exception code

    NOT_FOUND_PROJECT(77000, "프로젝트를 찾을 수 없습니다."),
    NOT_FOUND_INQUIRY(8000, "요청한 ID에 해당하는 문의가 존재하지 않습니다."),
    NOT_FOUND_INQUIRY_REPLY(8001, "요청한 ID에 해당하는 문의 답변이 존재하지 않습니다."),
    ALREADY_EXIST_INQUIRY_REPLY(8002, "이미 답변이 등록된 문의입니다."),

탈퇴한 회원 case

비고

  • 과제 제안 API 의 EmailService 동일하게 이용하여 메일 발송 기능 구현하였습니다
    • application-email.yml에 spring.mail.adminEmail 추가 필요
  • 문의 / 문의 답변이 수정된 경우 메일 발송 여부가 결정되어야 할 것 같습니다
  • 문의의 경우 조회 시 작성자를 username 으로 반환하도록 하였는데, userId 도 함께 반환하는 것이 더 적절한지 고려가 필요할 것 같습니다
  • inquiry 생성의 경우 projects/{projectId}/inquiry 를 통해 생성하지만 controllerTest 의 경우 임시로 InquiryControllerTest에 두었습니다

shj1081 and others added 12 commits August 21, 2024 16:32
* 과제 제안 API PR의 mail 관련 설정 참고
* fix: redirect uri 추가

* fix: access-token cookie응답으로 변경

* fix: cookie domain localhost로 변경

* fix: redirect uri 방식 변경

* fix: redirect uri 방식 변경

* fix: secure 옵션 추가

* fix: 프론트 로그인 콜백주소 변경

* chore: config typo

* refactor: Login 테스트코드 리팩토링

* refactor: 학번 uniqueness 삭제
@shj1081 shj1081 self-assigned this Aug 23, 2024
@shj1081 shj1081 added 🚀 Feature 기능 개발 🎯 API API 개발 관련 Inquiry Domain labels Aug 23, 2024
@shj1081 shj1081 requested review from jcy0308 and simta1 August 23, 2024 12:51
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.

  • 문의/문의 답변이 수정된 경우 메일 발송 여부 -> 일단은 지금처럼 두고 추후 요청이 있을 경우 수정하는걸로 하죠!
  • 문의의 경우 조회 시 작성자를 user 의 name 으로 반환하도록 하였는데, userId 도 함께 반환하는 것이 더 적절한지 고려가 필요할 것 같습니다 -> 현재 피그마상으로는이름만 있어도 될 것 같습니다.

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.

develop 브랜치 머지하고
html 파일들 빌드해서 커밋하고
최종 머지하면 될 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 API API 개발 관련 🚀 Feature 기능 개발 Inquiry Domain
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants