경북대 FE_정서현 6주차 과제 Step 3~4 #104
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
sjoleee
approved these changes
Aug 5, 2024
Comment on lines
+47
to
+52
| try { | ||
| const response = await fetchInstance.post(`${BASE_URL}/api/orders`, orderData, { | ||
| headers: { | ||
| "Content-Type": "application/json", | ||
| }, | ||
| }); |
| }, | ||
| }); | ||
|
|
||
| if (response.status === 200) { |
Comment on lines
+24
to
+31
| const [totalPrice, setTotalPrice] = useState(initTotalPrice); | ||
| const { handleSubmit, getValues, register, watch } = useOrderFormContext(); | ||
|
|
||
| const point = watch("point", 0); | ||
|
|
||
| useEffect(() => { | ||
| setTotalPrice(initTotalPrice - (point ?? 0)); // 포인트가 undefined일 경우 0으로 처리 | ||
| }, [point, initTotalPrice]); |
There was a problem hiding this comment.
totalPrice 상태와 useEffect는 제거할 수 있지 않을까요?
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.
📌 6주차 질문
🤔 1. SPA 페이지를 정적 배포하려고 할 때 Vercel 을 사용하지 않는다면 어떤 방법이 있을까요?
🤔 2. CSRF 나 XSS 공격을 막는 방법은 무엇일까요?
🤔 3. 브라우저 렌더링 원리에 대해 설명해주세요.
6주차가 개인적으로 가장 어려웠던 것 같습니다.
실제 백엔드 분들이 배포해주신 API 들을 이용하여 제대로 한번 구현해보고자 했으나,
백엔드 쪽에선 이미 관련 설정을 다 해주셨다고 했는데, 어떤 이유인지 저는 CORS 문제나 상기 문제들이 계속 발생하여 실제로 동작하는지 확인 해 볼 수 가 없었습니다.
그래서 MSW 로만 관련 기능을 테스트 하였습니다 ㅠㅠ
계속 시도해본 결과 로그인과 회원가입 API 통신은 정상적으로 작동하도록 구현했습니다!
다른 API 도 손보고 커밋 올려보겠습니다.