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

[feature/extractCouponCart] 장바구니 쿠폰 적용 알고리즘 변경 #133

Merged
merged 6 commits into from
Nov 15, 2022

Conversation

jungeu1509
Copy link
Member

@jungeu1509 jungeu1509 commented Nov 14, 2022

개요

장바구니 쿠폰 적용 알고리즘 변경

작업한 내용

  • 쿠폰 보는 알고리즘 제거를 위한 최적화
  • 계산은 백에서하고 프런트에선 최대한 보여주기만 하자
  • 모달 뒷편 회색 배경 추가

리뷰 가이드

페이지 멈추는지 안멈추는지 확인해주세요...

Lotte-Feelmycode/the-parabole-market#217 백엔드는 이 브랜치를 적용해야합니다.

테스트 결과

image

이슈번호

[#132]

@jungeu1509 jungeu1509 added the Fix 버그 수정 label Nov 14, 2022
@jungeu1509 jungeu1509 self-assigned this Nov 14, 2022
@jungeu1509 jungeu1509 linked an issue Nov 14, 2022 that may be closed by this pull request
chd830
chd830 previously approved these changes Nov 15, 2022
@daeunchung
Copy link
Contributor

daeunchung commented Nov 15, 2022

조금 이번 PR 과 무관할 수 있는데
장바구니에 물건 없어서 담으려는데 이미 담겼대요
image
모든 상품 다 해보는 중인데 전부 다 이미 담겼대요

BE branch는 바꿔서 적용했어요

@daeunchung
Copy link
Contributor

장바구니가 생기지 않아서 발생한 문제로 해결되었음

@daeunchung
Copy link
Contributor

image
정액 / 정율 type 을 UI 에 추가해주면 최고겠네요 고생많으셨어요

@jungeu1509
Copy link
Member Author

image

@daeunchung
Copy link
Contributor

LGTM 👍

@jungeu1509 jungeu1509 merged commit 51aa633 into dev Nov 15, 2022
@jungeu1509 jungeu1509 deleted the feature/extractCouponCart branch November 15, 2022 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 버그 수정
Projects
Development

Successfully merging this pull request may close these issues.

장바구니 오류 수정
3 participants