Conversation
src/app/shop/[category]/[productId]/_components/Product/OptionWithButtons.tsx
Outdated
Show resolved
Hide resolved
src/app/shop/[category]/[productId]/_components/Product/OptionWithButtons.tsx
Outdated
Show resolved
Hide resolved
| @@ -85,7 +85,7 @@ export default function Header() { | |||
| <SearchButton isBlack={isBlack} /> | |||
| {!users ? <LoginButton onClick={handleLoginButtonClick} /> : <LogoutButton />} | |||
| <button className={cn('user-icon')} type='button' onClick={handleUserIconClick}> | |||
| {profileImage ? ( | |||
| {profileImage && profileImage !== 'null' ? ( | |||
There was a problem hiding this comment.
| {profileImage && profileImage !== 'null' ? ( | |
| {profileImage && profileImage != null ? ( |
요즘 자바스크립트 공부하면서 드는 생각인데 이런식으로 null이 문자열로 넘어오는 경우에는 동치연산자 말고 동등 연산자로 사용해도 좋을거같아요
물론 컴포넌트 수정할때 제가 고려하지 못한 부분이지만.. 암튼
| thumbnail={thumbnail} | ||
| reviewscount={reviewCount} | ||
| isLiked={liked} | ||
| category={category} | ||
| hasShop={false} |
There was a problem hiding this comment.
이거 제가 옵셔널로 바꿔서 이제 false면 안적어도 될꺼에용! 이름이 같은건 ...rest로 할 수 없나요? ㅠ 진짜 프롭스가 넘 많네용,, 제가 만든 부분이라 제가 고민해봐야 할 거 같아요,,
There was a problem hiding this comment.
hasShop 말씀하시는걸까용?? 옵셔널로 안되어있는데..!! isLiked는 하트 버튼때문에 대부분 다 필요할거에요!?
| const handleClickBuyButton = () => { | ||
| checkUserAndOptions(); | ||
| if (checkUserAndOptions()) return; |
There was a problem hiding this comment.
| if (checkUserAndOptions()) return; | |
| if (checkUserAndOptions()) { | |
| return; | |
| } |
| setCookie('orderId', response.data.toString()); | ||
| router.push(ROUTER.MY_PAGE.CHECKOUT); |
There was a problem hiding this comment.
orderId 쿠키에 저장하기로 했었나요...?(정말 기억 안남😆)
| if (userData?.data && productId) { | ||
| addRecentProduct(productId); | ||
| } |
There was a problem hiding this comment.
useEffect로 mutation 불러오는 게 개인적으로는 살짝 위험하다고 생각합니다(dependency 관련),
이 부분은 상점 페이지에서 해당 상품을 클릭했을 때 mutation 발생시키는 건 어떨지 제안해봅니다!
There was a problem hiding this comment.
좋은 것 같은데 그럼 productlist나 productitem에서 onclick이벤트가 필요한데 지금 둘 다 서버 컴포넌트로 되어있어서🥹 고민을 좀 해봐야겠네요우...
There was a problem hiding this comment.
클라이언트 컴포넌트도 서버에서 prerender되기 때문에, prerender때 seo에 필요한 내용들은 반영되도록 하면 클라이언트 컴포넌트로 해도 큰 문제 없을 것 같아요
| price: number; | ||
| category: 'switch' | 'keyboard' | 'keycap' | 'etc'; | ||
| reviewCount: number; | ||
| liked: boolean; |
🚀 작업 내용
📝 참고 사항
🖼️ 스크린샷
🚨 관련 이슈 (이슈 번호)
✅ 체크리스트