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.

Sign up for GitHub

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

Jump to bottom

refactor: 도메인 변경 #6

Merged
merged 1 commit into from
Oct 7, 2023
Merged

refactor: 도메인 변경 #6

merged 1 commit into from
Oct 7, 2023

Conversation

kiljw316
Copy link
Contributor

@kiljw316 kiljw316 commented Oct 7, 2023 •
edited
Loading

Description

도메인 스펙 변경

References

Checklist

  • Add labels and reviewers ?
  • Are you fully explaining what you changed ?
  • Did you check that there is no conflict with the base branch ?

@kiljw316 kiljw316 self-assigned this Oct 7, 2023
@iamzin
Copy link
Contributor

iamzin commented Oct 7, 2023

  • 할인 정책 없음
  • 쿠폰 정책 없음
  • 옵션의 경우 아래와 같이 String 으로 처리
    옵션1: 블랙, M
    옵션2: 블랙, L

@iamzin
Copy link
Contributor

iamzin commented Oct 7, 2023

Product의 quantity - 재고 수량
Variant의 quantity - 구매 수량

@kiljw316 kiljw316 merged commit e6e93bf into main Oct 7, 2023
1 check passed
@kiljw316 kiljw316 deleted the refactor/domain branch October 7, 2023 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaesungahn91 jaesungahn91 jaesungahn91 approved these changes

@iamzin iamzin iamzin approved these changes

Assignees

@kiljw316 kiljw316

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kiljw316 @iamzin @jaesungahn91