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

도서 상세 페이지 이미지 표시 및 서버 도메인 모델 업데이트 #154

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Mar 6, 2024

작업 내용 (Content)

주요 변경 사항

  • 도서 상세 페이지에 이미지를 표시하도록 기능을 추가했습니다.
  • 서버 도메인 모델 Book 클래스에 imgId 속성을 추가했습니다.
  • findWithCategory 함수를 수정하여 이미지 ID 정보를 함께 조회하도록 했습니다.
  • 개발 서버 실행 스크립트를 tsx watch src/index.ts 에서 tsx watch -r dotenv/config src/index.ts 로 변경했습니다. 이는 .env 파일 변경 시에도 자동으로 서버를 다시 시작하도록 하기 위함입니다.

관련 변경 사항:

  • server/package.json 파일 수정
  • server/src/books/domain/books.repository.ts 파일 수정

링크 (Links)

기타 사항 (Etc)

Merge 전 필요 작업 (Checklist before merge)

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.
  • CI 파이프라인이 통과가 되었나요?
  • 리뷰 요청 전 Self-Review로 의문점을 해결 하셨나요?
  • PR 리뷰 가능한 크기를 유지하셨나요?

주요 변경 사항:
- 도서 상세 페이지에 이미지를 표시하도록 기능을 추가했습니다.
- 서버 도메인 모델 Book 클래스에 imgId 속성을 추가했습니다.
- findWithCategory 함수를 수정하여 이미지 ID 정보를 함께 조회하도록 했습니다.
- 개발 서버 실행 스크립트를 tsx watch src/index.ts 에서 tsx watch -r dotenv/config src/index.ts 로 변경했습니다. 이는 .env 파일 변경 시에도 자동으로 서버를 다시 시작하도록 하기 위함입니다.

관련 변경 사항:
- server/package.json 파일 수정
- server/src/books/domain/books.repository.ts 파일 수정
주요 변경 사항
- 도서 상세 페이지에 상세 설명과 목차 정보를 추가했습니다.
- 상세 설명은 텍스트 넘치는 경우 ellipsis를 활용하여 표시하도록 했습니다.
- EllipsisBox 컴포넌트를 추가하여 재사용 가능한 텍스트 넘치기 컴포넌트를 구현했습니다.

관련 변경 사항
- client/src/components/common/EllipisisBox.tsx 파일 생성 (새 파일)
- client/src/pages/BookDetail.tsx 파일 수정
@jihwooon jihwooon merged commit 2ada589 into main Mar 6, 2024
2 checks passed
@jihwooon jihwooon deleted the issue-82 branch March 6, 2024 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant