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

Dev #329

Open
wants to merge 659 commits into
base: main
Choose a base branch
from
Open

Dev #329

wants to merge 659 commits into from

Conversation

CessnaJ
Copy link
Contributor

@CessnaJ CessnaJ commented Apr 19, 2024

MR 이유

  • Feature 병합
  • 버그 수정(아래에 상세 기입)
  • 코드 스타일 변경 (formatting, local variables ex> indentation 변경, 변수명 변경, 주석 추가 )
  • 리팩터링 (로직변경 X, api 변경 X, ex> 함수 분리, 클래스 재구조화 )
  • 주기적인 master 병합
  • Build 관련 변경
  • CI 관련 변경
  • Docs 관련 변경
  • 기타(아래에 상세 기입)

스크린샷 또는 세부 내용

  • 세부사항을 항목으로 설명

MR 전 확인

  • 정상 작동
  • Conflict 발생 - 팀원과의 상의 필요

ssjy4974 and others added 30 commits November 1, 2023 15:44
add : 바로 구매하기 성공 시 라우터 추가
add : 바로 구매하기 console log 추가
fix : 바로구매하기 수정
fix : 바로구매주문시 stock 수정
add : addOrderSheet 콘솔 확인
✨Feat: 주문서에 결제금액 나오도록 처리
add : 관리자 로그인 리다이렉트, 로그인 하지 않은 유저 방어 로직 추가
✨ Faet : 주문에 주소 추가
✨FFix: ordersheet 가격 렌더링
qwerty1434 and others added 30 commits November 3, 2023 11:29
fix : 관리자 계정 수정 및 카트 0개 상품 주문시 방어 로직 추가
fix : 비로그인 카트 조회 방어 로직
fix : cart selectedQuantity 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants