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

Refactor : image 패키지 생성 & 프론트 요구사항 반영 #407

Conversation

GBGreenBravo
Copy link
Collaborator

@GBGreenBravo GBGreenBravo commented Mar 26, 2024

개요

  • 기존의 ImageStore/ImageReader은 global 패키지 하위에,
    ImageStoreImpl/ImageReaderImpl이 post 패키지 하위에 있어,
    이를 image 패키지를 생성하여 정리했습니다.

  • Image 조회/삭제 시, default-crew-icon이 정상 반영(기본 조회 / 삭제 면역)되도록 수정했습니다.

  • 그리고 아래의 금일 프론트 요구사항에 대해서도 반영했고, 빗금치지 않은 부분은 반영 안된 사항들입니다.
    image

  • (2024/04/01) 크루멤버목록 조회 API 구현 & 일정멤버목록 조회 API 리팩토링 진행했습니다.

closed #406

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제
  • 배포 및 PR 관련

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@GBGreenBravo GBGreenBravo added 🎸Chore 기타 변경사항 (빌드 스크립트 수정, 패키지 매니징 설정 등) 💻 BE 백엔드 🔨 Refactor 코드를 리팩토링한 경우 labels Mar 26, 2024
@GBGreenBravo GBGreenBravo self-assigned this Mar 26, 2024
@GBGreenBravo GBGreenBravo changed the title Refacto : image 패키지 생성 & 크루 default-image 반영 Refactor : image 패키지 생성 & 크루 default-image 반영 Mar 26, 2024
@GBGreenBravo GBGreenBravo changed the title Refactor : image 패키지 생성 & 크루 default-image 반영 Refactor : image 패키지 생성 & 금일 프론트 요구사항 반영 Mar 26, 2024
Copy link
Member

@c0olcode c0olcode left a comment

Choose a reason for hiding this comment

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

반영 내용 확인했습니다 감사합니다 :)

@GBGreenBravo
Copy link
Collaborator Author

GBGreenBravo commented Mar 28, 2024

내 가입/신청 크루 목록 조회 API를 구현할 때,
JoinedTime이나 AppliedTime을 어떤 레이어에서 결합하느냐에 따라 2가지 옵션이 있을 듯 합니다!

  1. Facade에서 결합
    • CrewsResponse를 구성할 때 인자로 JoinedTime/AppliedTime을 담기에, CrewDto에는 영향 없음.
    • 그러나 CrewFacade에서 CrewService 메서드를 여러번 호출해야 한다는 단점.
      (CrewDtos가져오고, 각 crewDto에 따른 가입/신청time조회)
  2. Service에서 결합
    • CrewDto 구성할 때, JoinedTime/AppliedTime을 담아야 하므로, CrewDto에 2개의 새 필드가 추가됨.
      (Aggregate 개념에 의해 비합리적이지는 않아보임.)
    • CrewDto에 다른 기능에서는 활용하지 않는 필드 늘어난다는 단점.
      (DDD강의의 Info 개념이 적용될 여지 있어보임)

@GBGreenBravo GBGreenBravo changed the title Refactor : image 패키지 생성 & 금일 프론트 요구사항 반영 Refactor : image 패키지 생성 & 프론트 요구사항 반영 Apr 1, 2024
Copy link
Collaborator

@oxix97 oxix97 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~~

@GBGreenBravo GBGreenBravo merged commit bc1a97c into Kernel360:develop Apr 9, 2024
1 check passed
@GBGreenBravo GBGreenBravo deleted the refactor/#406_Image_Package_&_Crew_Default_Image branch April 9, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 BE 백엔드 🎸Chore 기타 변경사항 (빌드 스크립트 수정, 패키지 매니징 설정 등) 🔨 Refactor 코드를 리팩토링한 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이미지 도메인 패키지 생성 & 크루 기본 이미지 반영
4 participants