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

feat: 어드민, 사장님 상점 생성, 수정시 메인 카테고리, 배너 이미지 추가 #1015

Merged
merged 10 commits into from
Nov 13, 2024

Conversation

krSeonghyeon
Copy link
Contributor

@krSeonghyeon krSeonghyeon commented Nov 12, 2024

🔥 연관 이슈

🚀 작업 내용

  • 어드민과 사장님 상점에서 생성, 수정시 메인 카테고리를 선택할 수 있는 기능 추가
  1. shops테이블에 메인 카테고리를 관리하기 위한 main_category_id 필드 추가
  2. 어드민, 사장님 상점 생성, 수정 DTO에 mainCategoryId 필드 추가
  3. 카테고리 테이블에 배너 이미지를 관리하기 위한 필드 추가
  4. 관련 테스트코드 수정

💬 리뷰 중점사항

  • 외래키 관계로 인해 테스트 코드 작성 시 많은 변화가 필요해서 리팩토링 기간에 외래키 작성에 대해 고민해보아야 할 것 같습니다

@github-actions github-actions bot added the 기능 새로운 기능을 개발합니다. label Nov 12, 2024
@krSeonghyeon krSeonghyeon self-assigned this Nov 12, 2024
Copy link

github-actions bot commented Nov 12, 2024

Unit Test Results

335 tests   334 ✔️  1m 35s ⏱️
  37 suites      1 💤
  37 files        0

Results for commit b0615d2.

♻️ This comment has been updated with latest results.

@krSeonghyeon krSeonghyeon force-pushed the feature/1005-admin-main-category branch from 4747c01 to bcf600c Compare November 13, 2024 01:43
@krSeonghyeon krSeonghyeon changed the title feat: 어드민, 사장님 상점 생성, 수정시 메인 카테고리 선택 기능 추가 feat: 어드민, 사장님 상점 생성, 수정시 메인 카테고리, 배너 이미지 추가 Nov 13, 2024
@krSeonghyeon krSeonghyeon merged commit ace969d into develop Nov 13, 2024
3 checks passed
@krSeonghyeon krSeonghyeon deleted the feature/1005-admin-main-category branch November 13, 2024 12:55
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.

상점에 따른 카테고리의 메인 카테고리 지정 선택 로직 추가
3 participants