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

week4 수정사항 #17

Open
23 of 24 tasks
Kdonghs opened this issue Sep 27, 2024 · 0 comments
Open
23 of 24 tasks

week4 수정사항 #17

Kdonghs opened this issue Sep 27, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@Kdonghs
Copy link
Contributor

Kdonghs commented Sep 27, 2024

공통

  • 주간 프로젝트 수행 완료후 코드 스타일 적용 바람

김동혁

김도헌

  • 프로젝트 조회시 로인한 유저의 프로젝트만 조회 가능하게 수정
  • 프로젝트 생성시 토큰내 정보의 유저로 프로젝트 생성
  • ProjectCreate, ProjectUpdate에서 guests는 불필요함
  • ProjectCreate, ProjectUpdate에서 options은 List으로 수정
  • ProjectCreate, ProjectUpdate의 startDate, endDate 검사

권순호

  • @RequestBody는 DTO로 입력 형태 만들어서 받을 것
  • SingleResult로 만들어서 값을 반환할 것(텍스트를 보내는건 예외처리시 하는 것)
  • @Valid로 입력값을 검증 할 것
  • deleteMember시 쿼리(리포지토리)로 지우지 말 것(조회해서 멤버가 존재하는지 검증하고 .delete(entity)로 삭제할 것)
  • addMember 다시 구현할 것(DTO에서 입력값을 검증하고, 프로젝트가 존재하는지 검증, 멤버 객체 생성(mapper로), save(entity))
  • add라는 명칭보다 create를 사용할 것(get,getList,create,update,delete)
  • updateMember다시 구현할 것(DTO에서 입력값을 검증하고, 멤버가 존재하는지 검증, 멤버 객체 수정(mapper로), save(entity) or @transactional)
  • @Autowired는 생성자에 사용할 것
  • 예외처리는 BaseHandler를 사용할 것
  • 조회기능도 구현할 것(get, getList)
  • MemberEntity에 taskEntities를 카스케이드 하면 안됨

조서영

  • isDeleted는 삭제 여부를 검사하기 위한 파라미터로 생성시 생성자에서 0으로 생성하면 됨(입력받지 말아야 함)
  • progress는 생성시 무조건 0이여서 입력받지 않아야 함
  • ownerId는 로그인한 토큰 내 정보를 파싱해서 사용함(UserService참조)
  • TaskEntity생성시 mapper로 구현하는게 가독성이 좋음
  • GuestEntity는 존재한다고 가정하고 구현 바람(mock데이터는 테스트에서나 사용하는 것)
  • swagger 주석 바람
  • 구현사항 README에 반영 바람
  • deleteTask의 반환값 추가 바람(SingleResult)
@seoyoung-dev seoyoung-dev added the enhancement New feature or request label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants