-
Notifications
You must be signed in to change notification settings - Fork 0
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: add banned word when create item #160
Conversation
backend/streetdrop-api/src/main/java/com/depromeet/domains/item/service/ItemService.java
Outdated
Show resolved
Hide resolved
backend/streetdrop-api/src/main/java/com/depromeet/domains/item/service/ItemService.java
Outdated
Show resolved
Hide resolved
backend/streetdrop-api/src/main/java/com/depromeet/common/repository/BannedWordRepository.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good
IOS 팀과 논의한 결과 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
놀러왔어요! 이 서비스 너무 좋아용~
backend/streetdrop-api/src/main/java/com/depromeet/common/repository/BannedWordRepository.java
Outdated
Show resolved
Hide resolved
6fe94f3
to
b0e959c
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## dev #160 +/- ##
============================================
- Coverage 18.16% 16.85% -1.32%
Complexity 49 49
============================================
Files 44 46 +2
Lines 567 617 +50
Branches 15 18 +3
============================================
+ Hits 103 104 +1
- Misses 462 511 +49
Partials 2 2
|
@siyeonSon @yunyoung1819 |
backend/streetdrop-api/src/main/java/com/depromeet/common/annotation/NotBannedWord.java
Show resolved
Hide resolved
backend/streetdrop-api/src/main/java/com/depromeet/common/repository/BannedWordRepository.java
Show resolved
Hide resolved
backend/streetdrop-api/src/main/java/com/depromeet/common/repository/BannedWordRepository.java
Show resolved
Hide resolved
...eetdrop-api/src/main/java/com/depromeet/common/annotation/validator/BannedWordValidator.java
Outdated
Show resolved
Hide resolved
backend/streetdrop-domain/src/main/java/com/depromeet/common/entity/BannedWord.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트 코드 충돌나는 것 수정되시면 바로 머지해주셔도 될 것 같습니다. 고생 많으셨습니다.
Good!
* ✨ feat(api): add banned word validator * ✨ feat(api): add sql to flyway * ✨ fix(api): add fix query * ✨ feat(api) : add NotBannedWord annotation to ItemCreateRequestDto * ♻️ chore: change contentWord to contentWords * 🐛 fix: add access to noargsconstructor * 🐛 fix: fix test case and if null banned word is true
Resolve : #159