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

[refactor/#74] 배송비 계산 로직 재수정 및 콘솔 삭제 #75

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

maylh
Copy link
Collaborator

@maylh maylh commented Nov 28, 2024

📑 이슈 번호


✨️ 작업 내용

  • 배송비 계산하는 코드들이 관련 컴포넌트마다 작성되어 있어서, 해당 부분 수정했습니다 ! 상위 컴포넌트에서 한 번만 계산하고 props로 전달 받아서 사용하는 방식으로 수정했습니닷 ~~ 로직도 약간 수정했어용
  • 선택된 아이템 없을 때 배송비가 3000원으로 책정되어 있는게 어색해서 selected item 없을 경우 배송비 0원으로 뜨게 수정했습니다
  • 콘솔 삭제 .. 했습니다 더블체크 했어야 되는데 ㅠ 제성합니다 ~ ㅠㅠ

🌊 코멘트


📸 구현 결과

스크린샷 2024-11-29 오전 1 03 07

@maylh maylh self-assigned this Nov 28, 2024
Copy link

라벨 지정 제대로 되었는지, assignee 및 reviewer 지정했는지 체크 한 번씩 하기 🧞‍♂️ 고생했어요 다음 작업도 화이팅 !!!!!! 😍

Copy link
Collaborator

@ocahs9 ocahs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

깔끔!! LGTM!! 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 배송비 로직 재수정
2 participants