-
Notifications
You must be signed in to change notification settings - Fork 2
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
[release] v1.0.6 #282
[release] v1.0.6 #282
Conversation
MNG-21 : 미션 인증 시 문구 추가
MNG-21 : 미션 인증 문구 추가 API 문서 반영
MNG-22 : 진행 중 미션에서 인증여부까지 확인
MNG-22 : 반복미션 모아보기 인증 인원수 리턴 타입 변경
fix : 반복미션 모아보기 인증 인원수 리턴 타입 변경
MNG-22 : 미션 관련 스케쥴링 추가
MNG-20 feat: 모두의 인증 현황에 미션 인증 댓글 수 추가
앞으로 해야할거
|
import java.util.Optional; | ||
|
||
@Repository | ||
public interface MissionArchiveRepository extends JpaRepository<MissionArchive, Long>,MissionArchiveCustomRepository { | ||
|
||
@Query("select m from MissionArchive as m where m.member =: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.
이거는 후에 querydsl로 수정이 되는건가요? 아니면 수정이 이미 됐나요?
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.
후에 querydsl로 변경 하겠숩니다!
@Service | ||
@RequiredArgsConstructor | ||
@Transactional | ||
public class CreateMissionCommentUseCase { |
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.
이렇게 한 클래스에 너무 많은 책임이 있어서 고민이 많았는데 이런 경우 연쇄패턴을 적용하면 결합도를 낮출 수 있다고 하더라구요? 이거에 대해 고민 중입니다
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.
우아 좋습니다!
PR 타입
개요
앱스토어 기준 v1.0.6 버전
변경 사항
코드 리뷰 시 참고 사항
테스트 결과