-
Notifications
You must be signed in to change notification settings - Fork 7
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
[BE/FEAT] 회원수정 7일동안 안할 시 알림 보내기 기능 #312
Conversation
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~!!
sendEmitterUseCase.sendNotification( | ||
emitter, eventId, eventId, "EventStream Created. [memberId=" + memberId + "]"); |
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.
3번째 파라미터 eventId -> emitterId로 수정해야할 것 같아요
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.
여기서 emitterId가 eventId랑 같기때문에 궅이 emitterId를 만들어서 넣을 필요는 없어 보여요
|
||
@Service | ||
@RequiredArgsConstructor | ||
public class sendEmitterService implements SendEmitterUseCase { |
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.
"S"endEmitterService...
return Long.parseLong(eventId.split("_")[1]); | ||
} |
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.
parseLong 메서드는 원시형으로 타입을 반환해서 Long 타입을 반환한다면 valueOf 메서드를 사용하는 것이 좋아보입니다.
|
||
@UseCase | ||
public interface ConnectEmitterUseCase { | ||
SseEmitter connect(final String memberId, final String lastEventId); |
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.
인터페이스 이름에서 Connect한다는 거를 나타내고 있기도 하고 저희 메서드 네이밍 규칙에 맞추는 게 좋을 거 같은 데 어떻게 생각하시나요?
closes #294
📢 주요 변경사항