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] user API 마무리 #76

Open
wants to merge 11 commits into
base: feat/43-user-api/base
Choose a base branch
from
Open

Conversation

simta1
Copy link
Contributor

@simta1 simta1 commented Aug 23, 2024

작성자: @simta1

#43

체크 리스트

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

작업 내역

  • getUserFavorites, rdstdocs
  • getUserInquiries, restdocs
  • getUserProposals, restdocs
  • getDepartments, restdocs

비고

  • feat/43-user-api/2 브랜치가 머지되지 않아서 커밋 내역이 좀 깁니다. "getUserInquiries, getUserProposals 구현" 커밋부터 현재 브랜치의 커밋입니다.
  • intellij에서 파일들을 인식을 못하길래 ./gradlew clean build했더니 해결되긴 했는데, 이후 getUserInquiries와 getUserProposals의 restdocs를 생성할 때 responseFields snippet를 찾을 수 없다는 오류가 나고 있습니다. 제 노트북에서만 안되는 것인지 불확실해서, 한 번 확인해주시면 감사하겠습니다.
  • 해결했습니다 감사합니다~~

@simta1 simta1 added 🚀 Feature 기능 개발 🎯 API API 개발 관련 User Domain User 도메인 관련 labels Aug 23, 2024
@simta1 simta1 requested review from jcy0308 and shj1081 August 23, 2024 16:49
@simta1 simta1 self-assigned this Aug 23, 2024
@shj1081
Copy link
Contributor

shj1081 commented Aug 23, 2024

저의 경우에는 restDocs에 reponse-fields snippet이 정상적으로 표시되는 것 같습니다. generated-snippets의 하위 directory 를 모두 지우고 다시 build 해보시는게 좋을 것 같습니다 👍🏻

@yesjuhee
Copy link
Contributor

머지 타겟을 feat/43-user-api/2로 임시 설정해놓으면 커밋 내역을 좀 더 깔끔하게 할 수 있습니다!
feat/43-user-api/2 브랜치가 feat/43-user-api/base 브랜치로 머지되면 자동으로 타겟 브랜치가 재설정됩니다

@simta1 simta1 changed the title [FEAT] user API, restdocs 구현 [FEAT] user API 마무리 Aug 28, 2024
@shj1081
Copy link
Contributor

shj1081 commented Aug 30, 2024

수고하셨습니다! 저는 문제없어보입니다

@yesjuhee
Copy link
Contributor

늦은 리뷰 죄송합니다 ㅠ.ㅠ 마감 기한이 중간고사 직후로 잡혀서 빨리 작업하거나 중간 끝나고 바로 작업하는게 좋을 것 같습니다. 리뷰 사항은 공통적으로 "응답 형식"이 좀 더 구체화되어야 할 것 같습니다. 아마 이 API 처음 작업할 때 피그마가 구체적으로 나오지 않았던 것 같은데 피그마에 맞춰서 수정부탁드립니다 🙇 🙇‍♂️

  1. getUserFavorites의 경우 현재 develop 브랜치를 머지한 후 talk, job_interveiw, project 관련 다른 분들이 작업해두신 dto를 참고하거나 가져와서 사용하면 될 것 같습니다.
  2. getUserInquiresgetUserProposals의 경우는 현재 피그마에 맞춰서 "답변 여부" 필드 하나만 더 추가하면 될 것 같습니다.

@yesjuhee
Copy link
Contributor

수정 요구사항에서 이해가 안되는것 있으면 언제든 질문해주세용

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

Successfully merging this pull request may close these issues.

3 participants