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.

Sign up for GitHub

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

Jump to bottom

Feat/21 레시피 리스트 조회 기능 구현 #28

Merged
merged 19 commits into from
Mar 15, 2024
Merged

Feat/21 레시피 리스트 조회 기능 구현 #28

merged 19 commits into from
Mar 15, 2024

Conversation

sangwoonoel
Copy link
Contributor

@sangwoonoel sangwoonoel commented Mar 14, 2024 •
edited
Loading

Overview

  • 유저가 요리한 레시피 리스트를 조회하는 기능을 구현했습니다.
  • 유저가 추천 받은 레시피 리스트를 조회하는 기능을 구현했습니다.

Change Log

To Reviewer

  • poetry에 pymongo를 추가했습니다.

Issue Tags

@sangwoonoel
build: poetry에 pymongo 추가
11d5c14 
# 21
@sangwoonoel
feat(controller): 유저가 요리한 레시피 목록 조회 API 구현
0da6634 
# 21
@sangwoonoel
feat(service): 유저가 요리한 레시피 목록 조회를 위한 기능 구현
7b8e62a 
# 21
@sangwoonoel
feat(entity): 유저가 요리한 레시피 목록 조회를 위한 entity 생성
094fd14 
# 21
@sangwoonoel
feat(repository): 유저가 요리한 레시피 목록 조회를 위한 데이터베이스 접근 코드 구현
7c32c5b 
# 21
@sangwoonoel
feat(utils): MongoDB의 ObjectID 자료형 처리를 위한 PyObjectId 자료형 생성
a32b5b7 
# 21
@sangwoonoel
feat(entity): Ingredient, Ingredients entity 생성
bfb0601 
# 21
@sangwoonoel
refactor(entity): 유저 정보를 조회하는 함수 삭제
cfa527a 
# 21
@sangwoonoel
feat(dto): 유저가 요리한 레시피 정보 반환을 위한 response 클래스 구현
cd07afe 
# 21
@sangwoonoel
test(test): 유저가 요리한 레시피 목록 조회 테스트 코드 작성
65687e9 
# 21
@sangwoonoel
fix(controller): 유저가 요리한 레시피 목록 조회 API 반환 값 타입 변경
4323e17 
# 21
@sangwoonoel
fix(repository): select_ingredients_by_ingredients_id 함수의 return 값 ty…
88bf840 
…pe hint 수정

# 21
@sangwoonoel
test(test): 재료 조회 테스트 코드 추가
ac51e4d 
# 21
@sangwoonoel
feat(service): user_id로 recipes_id를 함께 조회하던 로직을 user 조회와 recipes_id 조…
0520965 
…회로 분리

# 21
@sangwoonoel
fix(dto): API 응답에 유저가 요리한 recipes_id와 next_page_url 추가
8b030d6 
# 21
@sangwoonoel
fix(entity): 유저가 추천받은 레시피 조회를 위한 메소드 생성
dff9dbb 
# 21
@sangwoonoel
feat(controller): 유저가 추천 받은 레시피 조회 API 구현
f9994a4 
# 21
@sangwoonoel
refactor(controller): 유저가 추천 받은 레시피와와 요리한 레시피의 아이디를 recipes_id에 함께 저장…
a0fd567 
…해서 전송하도록 코드 수정

# 21
@sangwoonoel
test(test): 유저가 추천 받은 레시피 목록 조회 API 테스트 작성
8d60b83 
# 21
@sangwoonoel sangwoonoel requested a review from GangBean March 14, 2024 09:47
@sangwoonoel sangwoonoel self-assigned this Mar 14, 2024
sangwoonoel
sangwoonoel commented Mar 15, 2024
View reviewed changes
backend/app/api/routes/recipes/service/recipes_service.py
return recipes


def get_ingredients_list_by_recipes(recipes: Recipes) -> List[Recipes]:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

이 부분 List[Ingredients]로 수정해야합니다

GangBean
GangBean approved these changes Mar 15, 2024
View reviewed changes
Copy link
Contributor

@GangBean GangBean left a comment

Choose a reason for hiding this comment

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

많은 기능 구현하느라 고생하셨습니다 👍 👍

backend/app/api/routes/recipes/controller/recipes_controller.py

@recipes_router.get(
"/users/{user_id}/recipes/cooked",
response_description="유저가 요리한 레시피 목록"
Copy link
Contributor

Choose a reason for hiding this comment

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

description 👍

backend/app/api/routes/recipes/controller/recipes_controller.py
recipes = get_recipes_by_recipes_id(user_cooked_recipes_id)
# 레시피로 Ingredients 리스트 조회
ingredients_list = get_ingredients_list_by_recipes(recipes)
return GetRecipesReponseList(recipes, ingredients_list)
Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM 💯

backend/app/api/routes/recipes/controller/recipes_controller.py
# 레시피로 Ingredients 리스트 조회
ingredients_list = get_ingredients_list_by_recipes(recipes)
# 유저가 요리한 요리 표시를 위한 user_cooked_recipes_id 함께 넘기기
return GetRecipesReponseList(recipes, ingredients_list, user_cooked_recipes_id)
Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM 💯

backend/app/api/routes/recipes/dto/get_recipes_reponse.py


class GetRecipesReponse(BaseModel):
id: PyObjectId
Copy link
Contributor

Choose a reason for hiding this comment

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

PyObjectId 를 따로 정의해 mongodb Object_id와 동일한 개념을 만든게 좋습니다

backend/app/api/routes/recipes/dto/get_recipes_reponse_list.py
class GetRecipesReponseList:
recipe_list: List[GetRecipesReponse]
cooked_recipes_id: List[str]
next_page_url: str
Copy link
Contributor

Choose a reason for hiding this comment

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

Response 별도 정의 좋습니다 👍

backend/app/api/routes/recipes/entity/recipe.py
recipe_url: str
portion: str
recipe_img_url: str
model_config = ConfigDict(
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

backend/app/api/routes/recipes/entity/ingredient.py
populate_by_name=True,
arbitrary_types_allowed=True,
json_schema_extra={
"example": {
Copy link
Contributor

Choose a reason for hiding this comment

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

NoSQL의 스키마를 문서화 할 수 있는 방법을 찾아 연결하면 좋을 것 같습니다

@GangBean GangBean merged commit ff24458 into main Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GangBean GangBean GangBean approved these changes

Assignees

@sangwoonoel sangwoonoel

Labels
backend
Projects
None yet
Milestone
No milestone
2 participants
@sangwoonoel @GangBean