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

[고도화] 해시태그 검색 기능 고도화 #70

Closed
4 tasks done
djkeh opened this issue Aug 16, 2022 · 0 comments · Fixed by #93
Closed
4 tasks done

[고도화] 해시태그 검색 기능 고도화 #70

djkeh opened this issue Aug 16, 2022 · 0 comments · Fixed by #93
Assignees
@djkeh
Labels
enhancement New feature or request
Milestone
게시판 서비스 고도화

Comments

@djkeh
Copy link
Owner

djkeh commented Aug 16, 2022 •
edited
Loading

해시태그 기능을 고도화한다.

  • 하나의 글이 여러 개의 해시태그를 저장할 수 있도록 만들기
  • 별도 입력 공간을 주지 않고, 본문에서 해시태그를 파싱해서 기록하기
  • DB에는 #을 뺀 문자열을 저장할 수 있게 하기
  • 해시태그에 링크를 삽입하기
@djkeh djkeh self-assigned this Aug 16, 2022
@djkeh djkeh added this to the 게시판 서비스 고도화 milestone Aug 16, 2022
@djkeh djkeh mentioned this issue Oct 1, 2022
Closed
djkeh added a commit that referenced this issue Oct 5, 2022
@djkeh
#70 - 해시태그 서비스 구현
5736af0 
게시글에 달린 해시태그를 관리하기 위한 기능을 구현.
독특하게도 해시태그의 저장을 다루는 메소드가 없는데
해시태그의 저장은 현재 비즈니스 로직의 구조 상
게시글이 저장, 수정될 때만 일어나는 기능으로
연관관계 매핑을 통해 게시글 서비스에서 모두 처리가 되기 때문.

* 해시태그 이름으로 해시태그 정보를 db 조회하기
* 본문에서 해시태그 파싱하기
* 게시글이 더 이상 없는 해시태그를 삭제하기
* TODO: 삭제 기능의 테스트를 강의 중에 만들지 않았다. 추가하면 좋을 듯.
djkeh added a commit that referenced this issue Oct 5, 2022
@djkeh
#70 - 게시글 서비스의 해시태그 기능 변경점 반영
bd127f5 
테스트의 변화는 #68 의 e9f36a4 에서 도메인 변경을 하면서
이미 구상하여 작업하였음

주로 게시글이 저장, 수정, 삭제될 때
해시태그가 함께 어떻게 움직여야 하는지를 묘사한다.

강의 진행하면서 도메인 변경이 얼마나 넓은 영역에
영향을 미치는지 보여주면서, 컴파일 에러를 해소하기 위해
일단 테스트 변경까지 모두 반영하게 되었다.

이런 과정을 거치면서
코드 작성 과정을 잘 구분하고 보여줄 시간이 부족하여
이런 커밋들이 발생했는데,
실전에서 이렇게 작업 내역을 나누면
커밋으로 이력을 추적하면서 볼 때 다소 불편할 것 같다.
또한 사이의 커밋이 테스트 실패하는
불완전한 상태로 진행된 것도 문제.

서비스의 변경은 테스트 변경과 함께 담길 수 있도록 하자.

* TODO: `getHashtags()`의 로직은 개선 가능해보임
djkeh added a commit that referenced this issue Oct 5, 2022
@djkeh
#70 - 게시글 컨트롤러의 해시태그 기능 변경점 반영
e62a484 
이 정보는 뷰에서 해시태그에 링크를 달 때
게시글의 검색 필터 정보로 쓰기 위해 필요하다.

* TODO: `searchTypes`에서 재활용할 수 있는 데이터이므로 더 나은 방법이 있을지 고민해보자.

테스트의 변화는 #68 의 e9f36a4 에서 도메인 변경을 하면서
이미 구상하여 작업하였음
djkeh added a commit that referenced this issue Oct 5, 2022
@djkeh
#70 - 뷰 템플릿의 해시태그 기능 변화 대응
ca708c0 
* 게시글 페이지, 게시판 페이지: 해시태그에 링크를 달고, badge 스타일을 추가
* 게시글 추가 페이지: 해시태그 입력칸 삭제
@djkeh djkeh mentioned this issue Oct 5, 2022
Merged
djkeh added a commit that referenced this issue Oct 5, 2022
@djkeh
Merge pull request #93 from djkeh/feature/#70-enhance-hashtag
1be09a0
@djkeh djkeh closed this as completed in #93 Oct 5, 2022
@devcys22 devcys22 mentioned this issue Dec 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@djkeh djkeh

Labels
enhancement New feature or request
Projects
패스트캠퍼스 - 10개 프로젝트로 완성하는 백엔드 웹개발
Archived in project
Milestone
게시판 서비스 고도화
Development

Successfully merging a pull request may close this issue.

[고도화] 해시태그 검색 기능 고도화 djkeh/fastcampus-project-board
1 participant
@djkeh