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

전남대 FE_ 이은진 5주차과제 #73

Open
wants to merge 16 commits into
base: eunjin210
Choose a base branch
from

Conversation

eunjin210
Copy link

안녕하세요 멘토님 5주차 과제 입니다 !
test 코드를 처음 짜보다보니, 환경 설정에서 오류가 나 orderForm을 제대로 마무리하지 못했습니다 ㅜ

스크린샷 2024-07-26 234927
위와 같은 오류가 나서 해결 못하고 pr 을 올렸는데, 더 고민해보겠습니다.
pr 확인 후 오류에 대해서 조언 주시면 감사하겠습니다 이번주도 잘 부탁드립니다 !

@eunjin210 eunjin210 changed the title 전남대 FE_ 이은진 5주차과제 Step1 전남대 FE_ 이은진 5주차과제 Jul 28, 2024
productId: string;
}

const WISHLIST_PATH = `${BASE_URL}/api/wishlist`;

Choose a reason for hiding this comment

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

api 함수가 다 같이 있는데 쪼개는게 좋을거 같아요.

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.

2 participants