feat(notification): 알림 일괄 삭제 API #158
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ssosee
reviewed
May 4, 2025
| return ResponseEntity.ok(BasicResponse.success()); | ||
| } | ||
|
|
||
| // TODO: 개발용 API -> 추후 제거 필요 |
Member
Author
There was a problem hiding this comment.
앗 @Profile이 빈 단위로 적용할 수 있는 설정으로 알고 있어서 따로 붙이진 않았는데, 아니면 아예 컨트롤러를 분리할까요?
Member
There was a problem hiding this comment.
오잉 method 단위도 되지 않나요?
@Target({ElementType.TYPE, ElementType.METHOD})
@Retention(RetentionPolicy.RUNTIME)
@Documented
@Conditional(ProfileCondition.class)
public @interface Profile {
/**
* The set of profiles for which the annotated component should be registered.
*/
String[] value();
}- ElementType.TYPE → 클래스, 인터페이스, enum, 애노테이션 등 타입 선언에 사용 가능
- ElementType.METHOD → 메서드에 사용 가능
Member
Author
There was a problem hiding this comment.
요게 제가 알기론 @Bean 이 붙은 메서드에만 적용이 가능한 것으로 알아요!! (링크)
Member
Author
There was a problem hiding this comment.
직접 테스트 해보았을때 dev 로 해놓고 local 로 실행해두 호출이 되네유 ㅜ^ㅜ
Member
There was a problem hiding this comment.
오 맞네요! Bean 메소드만 해당되는 군요!
그러면 기존에 있는 TestController 를 활용하면 좋을 것 같아요!
ssosee
reviewed
May 4, 2025
|
|
||
| @Operation(summary = "알림 제거", description = "회원에게 생성된 모든 알림을 제거") | ||
| @DeleteMapping("/notifications") | ||
| @Profile("dev") |
Member
There was a problem hiding this comment.
@Profile("dev") 이거 제거하면 될거 같아요!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 작업 내용
🔗 참고할만한 자료(선택)
💬 리뷰 요구사항(선택)