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

[release] v1.0.6 #282

Merged
merged 50 commits into from
May 13, 2024
Merged

[release] v1.0.6 #282

merged 50 commits into from
May 13, 2024

Conversation

seungueonn
Copy link
Member

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 업데이트
  • 기타 사소한 수정

개요

앱스토어 기준 v1.0.6 버전

변경 사항

코드 리뷰 시 참고 사항

테스트 결과

seungueonn and others added 30 commits March 15, 2024 15:41
MNG-21 : 미션 인증 시 문구 추가
MNG-21 : 미션 인증 문구 추가 API 문서 반영
seungueonn and others added 20 commits May 6, 2024 21:23
MNG-22 : 진행 중 미션에서 인증여부까지 확인
MNG-22 : 반복미션 모아보기 인증 인원수 리턴 타입 변경
fix : 반복미션 모아보기 인증 인원수 리턴 타입 변경
MNG-22 : 미션 관련 스케쥴링 추가
MNG-20 feat: 모두의 인증 현황에 미션 인증 댓글 수 추가
@seungueonn seungueonn requested a review from minsu20 May 13, 2024 14:40
@minsu20
Copy link
Member

minsu20 commented May 13, 2024

앞으로 해야할거

  1. 푸시알림 보내는 로직 결합도 낮추기
  2. 매개변수가 너무 많은 함수에 대해 줄이기

import java.util.Optional;

@Repository
public interface MissionArchiveRepository extends JpaRepository<MissionArchive, Long>,MissionArchiveCustomRepository {

@Query("select m from MissionArchive as m where m.member =:memberId" )
Copy link
Member

Choose a reason for hiding this comment

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

이거는 후에 querydsl로 수정이 되는건가요? 아니면 수정이 이미 됐나요?

Copy link
Member Author

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 {
Copy link
Member

Choose a reason for hiding this comment

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

이렇게 한 클래스에 너무 많은 책임이 있어서 고민이 많았는데 이런 경우 연쇄패턴을 적용하면 결합도를 낮출 수 있다고 하더라구요? 이거에 대해 고민 중입니다

Copy link
Member Author

Choose a reason for hiding this comment

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

우아 좋습니다!

@minsu20 minsu20 self-assigned this May 13, 2024
@minsu20 minsu20 self-requested a review May 13, 2024 14:54
@seungueonn seungueonn merged commit 5f2eb44 into release May 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants