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

해시태그 팔로우 구현, 검색 API 개선 #145

Merged
merged 9 commits into from
Mar 22, 2022
Merged

해시태그 팔로우 구현, 검색 API 개선 #145

merged 9 commits into from
Mar 22, 2022

Conversation

vectorch9
Copy link
Member

@vectorch9 vectorch9 commented Mar 20, 2022 •
edited by seonpilKim
Loading

해시태그 팔로우

해시태그 팔로우 구현

  • 팔로우, 언팔로우 API 구현
  • 피드에 팔로우한 해시태그의 게시물을 표시하는 것은 추가 작업필요

피드에 해시태그의 모든 게시물을 노출하는 것이 아닌 "인기 게시물"만 노출하는 것 같음

검색 API 개선

기존의 유저만 검색하던 API 삭제

해시태그 연관 검색어 목록 페이징 조회 API 제거

신규 검색 API

  • 검색 조회수(count)를 기준으로 상위 50개의 결과 반환
  • 이때, 검색어와 정확히 일치하는 해시태그와 유저(username기준)는 결과에 반드시 포함됨
  • 회원가입 시 search 테이블에 count가 0 인 상태로 회원 정보 추가
  • 회원탈퇴는 아직 구현하지 않아서 이후 추가 구현할 예정 회원탈퇴 API 구현 #144
  • 해시태그의 경우 search 테이블에 등록하는 로직이 아직 없어 쿼리를 통해 DB에 직접 insert하여 테스트 했음
    -> 해당 로직 구현 후 한번더 테스트 부탁드립니다 @seonpilKim
  • 검색어 앞에 #이 붙은 경우엔 해시태그만 검색됨

검색 조회수 증가 API

  • 해시태그, 유저 각각의 API로 구현했음
  • 검색 후 클릭시, 해당 프로필로 이동하기 전 검색 조회수 증가 API를 호출하면 됨

게시물-해시태그 등록 로직 업데이트

  • 새로운 해시태그를 등록하는 경우, SearchHashtag 엔티티 생성
  • 해시태그를 삭제해야하는 경우, SearchHashtag 엔티티 제거(batch delete)

해결 이슈

@vectorch9 vectorch9 added the ✈enhancement New feature or request label Mar 20, 2022
@vectorch9 vectorch9 requested a review from seonpilKim March 20, 2022 06:36
@vectorch9 vectorch9 self-assigned this Mar 20, 2022
seonpilKim
seonpilKim reviewed Mar 20, 2022
View reviewed changes
Copy link
Member

@seonpilKim seonpilKim left a comment •
edited
Loading

Choose a reason for hiding this comment

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

고생하셨습니다. 제가 작업할 부분은 이따 저녁에 수정하겠습니다.

  • 이슈들 PR에 연결해주세요~

src/main/java/cloneproject/Instagram/dto/search/SearchDTO.java Show resolved Hide resolved
src/main/java/cloneproject/Instagram/repository/search/SearchRepositoryQuerydslImpl.java Outdated Show resolved Hide resolved
src/main/java/cloneproject/Instagram/repository/search/SearchRepositoryQuerydslImpl.java Show resolved Hide resolved
seonpilKim and others added 4 commits March 21, 2022 00:34
@seonpilKim
Feat: 해시태그 연관 검색어 목록 페이징 조회 API 제거
6f129c7 
Resolve: #143
@seonpilKim
Feat: 게시물/댓글 업로드 or 삭제 API 로직 추가
8a091fa 
- 게시물/댓글 업로드 API: 해시태그 첫 생성 시, SearchHashtag 엔티티 생성
- 게시물/댓글 삭제 API: 해시태그를 삭제하는 경우, SearchHashtag 엔티티 제거

Resolve: #143
@vectorch9
Fix: 일치하는 검색어 처리 로직 수정
3c48b31 
- 검색어와 완전히 일치하는 해시태그, 유저네임의 처리 수정
    - 쿼리를 날리는 방식에서 List의 contains함수를 이용하도록 수정
- 검색 조회수 증가의 유저, 해시태그를 하나의 API로 합침
@vectorch9
Fix: 테스트 후 복구시켜놓지 않은 코드 수정
736b770
@vectorch9
Copy link
Member Author

일치하는 검색어 처리 로직 수정

  • 검색어와 일치하는 유저네임, 해시태그 처리 로직 수정
  • 쿼리를 통해 점검하는 방식 -> List의 contains 함수를 이용하는 방식

검색 조회수 증가 API 수정

  • 유저, 해시태그 각각의 API를 하나의 API로 합침

@vectorch9 vectorch9 merged commit cbe3053 into develop Mar 22, 2022
@vectorch9 vectorch9 deleted the Feature/HashtagFollow branch March 22, 2022 06:28
@vectorch9 vectorch9 changed the title Feature/hashtag follow 해시태그 팔로우 구현, 검색 API 개선 Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seonpilKim seonpilKim seonpilKim approved these changes

Assignees

@vectorch9 vectorch9

Labels
✈enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

연관 검색어 조회 API 추가, 검색 API 추가 검색에 해시태그 추가
2 participants
@vectorch9 @seonpilKim