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

Feature/#90 search api #94

Merged
merged 12 commits into from
Aug 15, 2024
Merged

Feature/#90 search api #94

merged 12 commits into from
Aug 15, 2024

Conversation

ShapeKim98
Copy link
Contributor

@ShapeKim98 ShapeKim98 commented Aug 14, 2024

#️⃣연관된 이슈

#90

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 검색 api를 연결하였습니다.
  • 최근검색어 저장기능을 구현하였습니다.

스크린샷 (선택)

Simulator Screen Recording - iPhone 15 Pro - 2024-08-14 at 15 34 05

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 꼭 이 기능에 대한건 아니긴 한데, 이러한 컨텐츠 목록에서 수정/추가하기로 이동 후 수정/추가한 내용을 어떻게 반영하면 좋을지 고민입니다.
    Simulator Screen Recording - iPhone 15 Pro - 2024-08-14 at 15 37 49
  • 페이지네이션은 아직 적용하지 않았습니다.
  • 아직 서버쪽에서 검색 api가 제대로 완성되지 않은건지, 검색 결과가 좀 이상하니 참고 바라겠습니다.(카테고리 필터가 안됨) 해결 완!

close #90

@ShapeKim98 ShapeKim98 added Feat 기능구현 Fix 기능 수정 API 🛜 기능구현 - API labels Aug 14, 2024
@ShapeKim98 ShapeKim98 self-assigned this Aug 14, 2024
@ShapeKim98 ShapeKim98 linked an issue Aug 14, 2024 that may be closed by this pull request
2 tasks
@stealmh
Copy link
Member

stealmh commented Aug 14, 2024

고생하셨습니다!
PR 천천히 읽어보겠습니다. 수정 / 추가한 항목을 반영하려면 직전화면이 어떤 화면인지 알아야할 것 같아요. delegate로 이전 화면에 추가한 혹은 수정된 항목을 update해주는 방식을 생각해두고 있는데 좀 복잡할 것 같아서 토요일에 다시 한번 논의 해보시죠

Comment on lines +220 to +225
.init(
page: pageable.page,
size: pageable.size,
sort: pageable.sort
),
true
Copy link
Member

Choose a reason for hiding this comment

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

init으로 바로 때려박는거보다 request라는 상수하나 만들고 인자넣는방식이 더 좋아보이긴합니다! true가 너무 불쌍하게 떨어져있음ㅠ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵!

Copy link
Member

@stealmh stealmh left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다! 검색어관련해서 userdefaults도 잘 구현해주신 것 같아요! 크게 문제되는 부분은 없으나 여전히 case let이 사용되지 않은 부분이 많이 보입니다. 일단 approve는 하겠지만 보이면 다음 PR때 수정해주세엽~~!

@stealmh stealmh merged commit 0ec6f18 into develop Aug 15, 2024
1 check passed
@stealmh stealmh deleted the feature/#90-search-api branch September 3, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API 🛜 기능구현 - API Feat 기능구현 Fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

검색 기능 api 연결
2 participants