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

Week6 조서영 기능 구현 #49

Merged
merged 67 commits into from
Oct 11, 2024

Conversation

seoyoung-dev
Copy link
Contributor

@seoyoung-dev seoyoung-dev commented Oct 10, 2024

서비스 플로우에 태스크 기능 잘 동작하는지 확인했습니다.
swagger 문서도 잘 나오는 것 확인했습니다.

주요 변경사항

  • response에 무한 depth 생성 오류
    • TaskDetail로 응답할 수 있도록 refactoring, mapper이용

  • TaskRespository의 findAllByProjectIdAndIsDeletedFalse 메서드를 findAllByProjectEntityIdAndIsDeletedFalse로 변경
    • 프로젝트 아이디로 태스크 리스트 조회시 발생하는 에러 해결
    • issue #47

  • 태스크 생성시 날짜 validation 로직 수정
    • 기존 코드는 프로젝트의 startDate가 태스크의 startDate 이후에 생성되었는지 검증함(반대가 되어야 함)
    • issue #44

해결하지 못한 이슈

  • 테스트 코드에서 project 생성시 id값이 자동으로 추가 되지 않음
  • projectId로 저장된 엔티티를 조회할 수 없기 때문에 태스크 생성 기능 동작 불가
  • 멘토링 시간에 문의해 볼 예정

Kdonghs and others added 30 commits September 27, 2024 22:01
w4 Develop2Master
sunandrabbit and others added 28 commits October 4, 2024 17:18
김도헌 week5
# Conflicts:
#	src/main/java/team1/BE/seamless/entity/ProjectEntity.java
#	src/main/java/team1/BE/seamless/entity/TaskEntity.java
#	src/main/java/team1/BE/seamless/init/InitData.java
#	src/main/java/team1/BE/seamless/init/ProjectCreator.java
#	src/main/java/team1/BE/seamless/repository/ProjectRepository.java
#	src/main/java/team1/BE/seamless/service/UserService.java
# Conflicts:
#	src/main/java/team1/BE/seamless/entity/MemberEntity.java
#	src/main/java/team1/BE/seamless/service/MemberService.java
#	src/main/java/team1/BE/seamless/service/TaskService.java
@seoyoung-dev seoyoung-dev changed the title Week6 기능 구현 Week6 조서영 기능 구현 Oct 10, 2024
@Kdonghs Kdonghs merged commit 1739cf5 into kakao-tech-campus-2nd-step3:seoyoung Oct 11, 2024
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.

4 participants