Skip to content

[7주차/유리] 워크북 제출합니다#40

Open
Kohseoyoung wants to merge 1 commit into
UMC-Inha:yuri/mainfrom
Kohseoyoung:main
Open

[7주차/유리] 워크북 제출합니다#40
Kohseoyoung wants to merge 1 commit into
UMC-Inha:yuri/mainfrom
Kohseoyoung:main

Conversation

@Kohseoyoung
Copy link
Copy Markdown

✅ 실습 체크리스트

  • 이론 학습을 완료하셨나요?
  • 미션 요구사항을 모두 이해하셨나요?
  • 실습을 수행하기 위한 공부를 완료하셨나요?
  • 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

@Kohseoyoung Kohseoyoung requested a review from YoungJJun May 14, 2026 02:05
@Kohseoyoung Kohseoyoung self-assigned this May 14, 2026
@Kohseoyoung Kohseoyoung linked an issue May 14, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@YoungJJun YoungJJun left a comment

Choose a reason for hiding this comment

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

7주차 피드백

  1. Member 리팩토링 잘 해주셨습니다.

  2. Mission 엔티티도 비어있었는데 잘 구현된 것 같아요.

  3. 미션이 아직 대부분 완료되지 않은 것 같아요!

    내가 진행중인 미션 조회 API, 내가 생성한 리뷰 조회 API, RequestBody Validation Check 및 General Exception에서 처리하도록 구현

    모두 필수 미션이어서 구현해주세요!

  4. MissionSuccessCode, MemberSuccessCode

    두 성공코드 모두 CREATED가 있는데 실제 HTTP 코드는 OK로 들어가있어요! HTTP 코드에도 CREATED가 있습니다.

    CREATED(HttpStatus.CREATED, "MISSION200_1", "성공적으로 미션을 생성했습니다.")

    이런식으로 수정해주세요.

  5. MissionErrorCode

    public enum MissionErrorCode implements BaseErrorCode{
        MEMBER_NOT_FOUND(HttpStatus.NOT_FOUND, "MEMBER404_1", "해당 미션을 찾을 수 없습니다.");
    
        private final HttpStatus status;
        private final String code;
        private final String message;
    }

    미션에러코드에 MEMBER_NOT_FOUND가 있습니다. MISSION 으로 바꿔주세요.

  6. Mission 엔티티에서 start_dt, end_dtString 타입으로 사용하고 있습니다.

    LocalDate 타입을 사용하면 날짜 유효성 검증과 날짜 연산이 훨씬 안전해집니다.

    → 동일하게 String으로 사용하신 부분 더 있으면 찾아서 수정해주세요! 제가 다 확인하지는 못한 것 같습니다.


유리 7주차 수고하셨습니다. 이번주는 아직 구현되지 않은 부분이 많은 것 같아요! 추가해주시고 미션 완료여부는 Swagger 캡쳐를 통해 확인할거라 우선 Approve 하겠습니당!

@YoungJJun
Copy link
Copy Markdown
Collaborator

아 추가로 API 구현이 많이 어려우시거나 구현했는데 해당 사항에 대해 추가로 피드백이 필요하시면 디코로 알려주세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter07_API 설계 심화 - 페이징

2 participants