경북대 FE_이정민 6주차 과제 Step3 #84
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 22 commits
August 4, 2024 19:27
userjmmm
force-pushed
the
step3-userjmmm
branch
from
342494d to
bba4c08
Compare
sjoleee
approved these changes
Aug 5, 2024
There was a problem hiding this comment.
수고하셨습니다~
코드는 step1, 2 리뷰를 참고해 주시면 좋을 것 같아요.
현재 여러 요청을 순차적으로 보내는 방식(waterfall)을 사용하고 계신데, 이 방식은 상품의 수가 많아질수록 오래 걸리기 때문에 좋은 방법이라고 보기는 어려울 것 같아요.
그리고 만약 특정 상품만 주문에 실패한다면 그것도 조금 이상하지 않을까요?
예를 들면 식빵이랑 잼을 사서 같이 먹고싶은 사람인데 식빵만 실패해서 잼만 먹어야 하는 상황을 생각해보면...
차라리 모두 실패시키고 다시 주문하는 것을 유도하는 방식이 안전하지 않을까 싶기도 하네요!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 멘토님,
상품 구매에 사용할 수 있는 포인트 기능을 추가한 코드입니다.
저희 조는 포인트를 사용해서 결제가 되는데, 이때 총 금액에서 5% 할인된 가격으로 결제되도록 기능을 구체화했습니다.
여러 옵션이 선택되었을 경우, 옵션마다 post 요청을 보내되 직렬 처리를 했습니다. 따라서 여러 개를 구매해도 전체 포인트에서 요청마다 하나씩 point를 감소시키고 다음 요청을 처리하는 방식으로 진행되도록 했습니다!
해당 로직은
src/components/features/Order/OrderForm/OrderInfo/index.tsx에서 확인 가능합니다.감사합니다 :)