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: 취향 테스트, 레시피 관련 기능 개발 #17

Merged
merged 7 commits into from
Aug 10, 2024
Merged

Conversation

Jeongh00
Copy link
Member

@Jeongh00 Jeongh00 commented Aug 9, 2024

🌍 이슈 번호

resolves #8
resolves #7

📝 구현 내용

  • 음식 추천 테스트 저장, 조회 API
  • 조리법 결과 Open API 결과 조회 API
  • Ingredient 파일 헥사고날 아키텍쳐에 맞게 패키지 변경
  • application-openapi-key.yml 로 시크릿 키 분리 (노션에 파일 저장)
  • quisine -> cuisine 으로 패키지 오타 수정

🍀 확인해야 할 부분

  • ingredient 관련 패키지들 제가 생각하는 구조에 맞게 변경해 두었습니다. 참고해서 다음 작업에 이대로 작업해주세요 ^^
  • 공통 Response type 소정 : ApiResponse / 정후 : ApplicationResponse 로 사용중인데 통일 필요
  • 예외처리 도 open api 부분, RuntimeException 부분 수정 부탁드려요 !!
  • 저도 소정님과 같이 WebClient 로 open api 호출했는데 , 공통 형식으로 끌어다 쓰는걸로 수정할지, Open Feign 으로 다시 수정할지 고민 좀 해보겠습니다.

@Jeongh00 Jeongh00 added bug Something isn't working enhancement New feature or request api Further information is requested labels Aug 9, 2024
@Jeongh00 Jeongh00 requested a review from xxoznge August 9, 2024 12:37
@Jeongh00 Jeongh00 self-assigned this Aug 9, 2024
@xxoznge xxoznge changed the title Feature/#8 feat: 취향 테스트, 레시피 관련 기능 개발 Aug 10, 2024
@xxoznge
Copy link
Collaborator

xxoznge commented Aug 10, 2024

pr 제목 수정하였습니다!

@Jeongh00 Jeongh00 merged commit 55440fa into develop Aug 10, 2024
1 check passed
@Jeongh00 Jeongh00 deleted the feature/#8 branch August 10, 2024 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Further information is requested bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants