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

[스프린트 3] 강수민 - 팀 생성 기능 추가 #31

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

wvssm
Copy link
Contributor

@wvssm wvssm commented Dec 2, 2024

Resolves #21

❓ 해결하려는 문제가 무엇인가요?

  • 팀 생성 기능을 만든다.

🛠️ 어떻게 해결했나요?

  • Valid 어노테이션을 사용하여 입력값을 검증했다.
  • TeamMember 테이블을 만들어 다대다 관계를 만들었다.
  • 팀 생성자를 일단 팀 매니저로 임명했다. 한 명의 회원이 여러 팀의 회장일 수 있으므로, 일대다 관계로 설정했으며, 단방향으로 설정하였다.

🔍 어떤 부분에 집중하여 리뷰해야 할까요?

  • TeamRequest Dto 코드가 난잡해서 리팩토링이 필요할 것 같다.
  • 마찬가지로 TeamService의 createTeam도 함수길이가 길어 메서드 분리가 필요할 것 같아서 체크하면 좋을 것 같다.

📚 참고 자료


📋 RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@wvssm wvssm changed the title Develop [스프린트 3] 강수민 - 팀 생성 기능 추가 Dec 2, 2024
@wvssm wvssm added the 기능 새로운 기능 추가나 요청 label Dec 2, 2024
private Member teamManager;

@OneToMany(mappedBy = "team")
private List<TeamMember> teamMembers = new ArrayList<>();

Choose a reason for hiding this comment

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

Fetch Lazy 설정 필요

private String teamPassword;

@CreatedDate
private LocalDateTime createdAt;

Choose a reason for hiding this comment

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

createDate 는 설정 따로 빼기 (상속받기)

return name;
}

public static Topic fromName(String name){

Choose a reason for hiding this comment

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

stream 쓰기

@ddglackrp
Copy link

다음에 수정 부탁드립니다~~

@ddglackrp ddglackrp merged commit 61bd825 into PNUMeat:develop Dec 2, 2024
@wvssm
Copy link
Contributor Author

wvssm commented Dec 2, 2024

다음에 수정 부탁드립니다~~

넵! 다음 스프린트 때 수정하겠습니다!

@wvssm wvssm self-assigned this Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능 추가나 요청
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants