경북대 FE_김민주 6주차 과제 Step 3, 4 #107
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
- 간단한 함수 컴포넌트화하지 않고 직접 사용 - 로고 클릭하면 홈으로 연결 - 회원가입 페이지에서 로그인 페이지로 연결하는 버튼 추가 - 에러텍스트 디자인 변경
- useMutation 인자로 mutationFn 넣고 오류 픽스
- 로그인과 회원가입 로직, 요청/응답이 같아서 useRegister -> useMembers로 수정하고 엔드포인트 인자를 받아 요청 url 달리 하도록 설정 - 토큰 리턴할 때 객체로 리턴하도록 - Home/MyAccount에 authToken 안 보이게 수정 필요 - 헤더에 로그인/내 계정 글자 바로 안 바뀌는 것 수정 필요
- Auth - 임시로 정해둔 사용자 이름 등등 바꿈 - Storage - emailSessionStorage 추가 (이메일 getter, setter) - 화면에 표시되는 사용자 이름 token -> email 변경 - MyAccount에 로그아웃 시 이메일 삭제 코드 추가
- 응답 객체 키 수정 (types에서 CategoryData 키 명 변경) - getCategories (통신하는 비동기 함수) 에러 처리 구문 추가 - 응답 객체 타입을 지정해놨는데 키 명이 안 맞아서 계속 오류 났었음
- types/ProductData 키 명 변경 및 추가 (api 명세에 맞춰서) - getProductDetail 함수에 에러 처리 구문 추가
- getProductOptions 함수에 에러 처리 구문 추가, export 삭제 - ProductOptionsData 객체 api 명세에 맞추어 키 명 수정
- currentApi를 로컬스토리지의 apiOption으로 가져오고 드롭다운의 디폴트 밸류로 설정 - 로컬스토리지에 apiOption이 없을 때는 "API 선택", 있을 때는 해당 값으로 나옴 -> 실제 로컬스토리지에 저장된 값과 화면에 뜨는 내용 일치
- 위시 등록하는 usePostWish 훅 구현 후 - useHandleWish에서 훅 호출
- api마다 option 응답 데이터의 키가 name/optionName로 다르게 설정되어 있어 모두 추가함
danmin20
requested changes
Aug 5, 2024
Comment on lines
+69
to
+76
|
|
||
| 1. 브라우저는 HTML을 파싱하여 DOM 트리를 만들고, CSS를 파싱하여 CSSOM 트리를 생성합니다. | ||
| 트리의 각 노드는 HTML 요소입니다. | ||
| 2. DOM 트리와 CSSOM 트리를 결합하여 렌더 트리를 생성합니다. | ||
| 3. 레이아웃: 화면에 페인팅할 요소들의 크기나 위치를 계산합니다. | ||
| 4. 페인팅: 각 노드를 화면에 그립니다. | ||
| 5. 컴포지팅: 최종적으로 화면을 브라우저에 띄웁니다. |
There was a problem hiding this comment.
리페인트, 리플로우 같은 사이드이펙트에 대해서도 같이 알아두시면 좋을 것 같아요~
Comment on lines
+13
to
+22
| export const BASE_URL = | ||
| apiOption === '김은선' | ||
| ? BASE_URL_KIM | ||
| : apiOption === '박준석' | ||
| ? BASE_URL_PARK | ||
| : apiOption === '안재민' | ||
| ? BASE_URL_AHN | ||
| : apiOption === '이도훈' | ||
| ? BASE_URL_LEE | ||
| : BASE_URL_KIM; |
There was a problem hiding this comment.
삼항연산자를 깊게 쓰면 가독성이 좋지 않아요~ key value 형싱으로 매핑해도 좋겠네요~
| ? BASE_URL_LEE | ||
| : BASE_URL_KIM; | ||
|
|
||
| console.log('BASE_URL: ', BASE_URL); // test |
| subtitle: string; | ||
| title: string; | ||
| amount: number; | ||
| } & React.HTMLAttributes<HTMLDivElement>; |
There was a problem hiding this comment.
저번에도 말씀드렸듯이, html엘리먼트 어트리뷰트를 모두 사용하시려는 이유가 궁금합니다~
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.
3단계 포인트는 UI 및 프론트 쪽 로직은 모두 구현했지만 api 요청 보내는 부분은 작성 못 했습니다.
4단계 질답은 모두 작성했습니다.