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(api): add search term recommend #327

Merged
merged 7 commits into from
Oct 10, 2023

Conversation

seonghun-dev
Copy link
Collaborator

@seonghun-dev seonghun-dev commented Sep 6, 2023

@codecov-commenter
Copy link

codecov-commenter commented Sep 6, 2023

Codecov Report

Merging #327 (c160df0) into dev (36f71bd) will decrease coverage by 0.56%.
Report is 1 commits behind head on dev.
The diff coverage is 11.76%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #327      +/-   ##
============================================
- Coverage     28.15%   27.59%   -0.56%     
  Complexity       76       76              
============================================
  Files            45       47       +2     
  Lines           611      627      +16     
  Branches         18       18              
============================================
+ Hits            172      173       +1     
- Misses          437      452      +15     
  Partials          2        2              
Files Coverage Δ
...java/com/depromeet/common/error/dto/ErrorCode.java 100.00% <100.00%> (ø)
...ecommend/controller/SearchRecommendController.java 0.00% <0.00%> (ø)
...ains/recommend/service/SearchRecommendService.java 0.00% <0.00%> (ø)

@seonghun-dev seonghun-dev added the ✨feature New feature or request label Sep 6, 2023
Copy link
Member

@DongGeon0908 DongGeon0908 left a comment

Choose a reason for hiding this comment

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

13기 프로젝트 보면서, 가장 인상 깊게 본 server-api인 것 같아요.
항상 응원하고, 염탐하고 있습니다. ㅎㅎ

화이팅~~~ cc. @yunyoung1819

@seonghun-dev seonghun-dev marked this pull request as ready for review September 29, 2023 03:01
@seonghun-dev
Copy link
Collaborator Author

@siyeonSon @yunyoung1819
배치쪽에 조금 공부가 필요해서 조회 가능한 우선 API랑 등록 가능한 Admin API 만 작업해두었습니다

@seonghun-dev seonghun-dev added 😺 API Server Api Domain 🥇asap As soon as possible labels Oct 3, 2023
Copy link
Collaborator

@yunyoung1819 yunyoung1819 left a comment

Choose a reason for hiding this comment

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

코멘트 달린 것 수정 완료되면 머지해주셔도 될 것 같아요 고생 많으셨습니다! ^^

@seonghun-dev seonghun-dev merged commit 4da3c11 into dev Oct 10, 2023
1 check passed
@seonghun-dev seonghun-dev deleted the feature/music-recommend branch October 10, 2023 18:53
seonghun-dev added a commit that referenced this pull request Nov 8, 2023
* ♻️ feat(api): add search term recommend

* ✨ feat(api) : add textcolor vo and dto

* 🐛 fix(api) : add api to text color dto

* ♻️ chore(admin): fix to var

* ♻️ chore(admin): fix to builder pattern

* ♻️ chore(admin): fix to protected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😺 API Server Api Domain 🥇asap As soon as possible ✨feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create search term recommendation API
5 participants