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

[전준기] sprint9 #131

Open
wants to merge 1 commit into
base: next-전준기
Choose a base branch
from

Conversation

JeonJun02
Copy link
Collaborator

요구사항

  • Github에 위클리 미션 PR을 만들어 주세요.

  • React 혹은 Next.js를 사용해 진행합니다.

  • 프론트엔드에서 api 요청 구현은 TanStack React Query를 활용해 주세요.

  • API는 https://panda-market-api.vercel.app/를 사용합니다. 명세는 https://panda-market-api.vercel.app/docs를 확인해 주세요.
    로그인/회원가입 페이지

  • JavaScript로 구현한 로그인/회원가입 페이지를 React.js 혹은 Next.js로 마이그레이션해 주세요.
    로그인 페이지

  • "회원 가입하기"를 클릭하면 회원가입 페이지로 이동해 주세요.

  • 로그인 실패하는 경우, 이메일 input 아래에 "이메일을 확인해 주세요.", 비밀번호 input 아래에 "비밀번호를 확인해 주세요." 에러 메시지를 표시해 주세요.

  • 로그인 버튼이 활성화된 후, 로그인 버튼 클릭 또는 Enter키 입력으로 로그인 실행합니다.

  • "/auth/signIn"으로 POST 요청해서 성공 응답을 받으면 중고 마켓 페이지로 이동합니다. 참고로 JWT로 구현되어 있습니다.

  • 실패할 경우, 실패 메시지를 모달을 통해 표시합니다.
    회원가입 페이지

  • "회원 가입하기"를 클릭하면 '/signin' 페이지로 이동합니다

  • 회원가입 버튼 클릭 또는 Enter키 입력으로 회원가입을 실행합니다.

  • 비밀번호 input과 비밀번호 확인 input의 값이 다른 경우, 비밀번호 확인 input 아래에 "비밀번호가 일치하지 않아요." 에러 메시지를 표시해 주세요.

  • 버튼이 활성화된 후, 회원가입은 "/auth/signUp" POST 요청해서 진행합니다. 참고로 JWT로 구현되어 있습니다.

  • 회원가입 성공 응답을 받으면 중고마켓 페이지로 이동합니다.

  • 실패할 경우, 실패 메시지를 모달을 통해 표시합니다.
    로그인, 회원가입 페이지 공통

  • 눈 모양 아이콘 클릭 시 비밀번호의 문자열이 보이기도 하고, 가려집니다.

  • 비밀번호의 문자열이 가려질 때는 눈 모양 아이콘에는 사선이 그어져 있고, 비밀번호의 문자열이 보일 때는 사선이 없는 눈 모양 아이콘이 보입니다.

  • 소셜 로그인에 구글 아이콘 클릭 시 'https://www.google.com/', 카카오 아이콘 클릭 시 'https://www.kakaocorp.com/page'로 이동합니다.

  • 로그인/회원가입 시 성공 응답으로 받은 accessToken을 로컬 스토리지에 저장합니다.

  • 로그인/회원가입 페이지에 접근 시 로컬 스토리지에 accessToken이 있는 경우 '/items' 페이지로 이동합니다.
    GNB

  • 상단 내비게이션 바에 프로필 영역은 인가된 경우, 유저 정보 API를 활용해 주세요.

  • 인가되지 않았을 경우 "로그인" 버튼이 보이게 해 주세요.
    상품 상세 페이지

  • PC, Tablet, Mobile 디자인에 해당하는 상품 상세 페이지를 만들어 주세요.

  • 상품 상세 페이지 url path는 "/items/{itemId)"로 설정하세요.

  • '목록으로 돌아가기' 버튼 클릭 시 중고마켓 페이지 "/items"로 이동합니다.

  • 상품 상세 데이터는 '/products/{productId}' GET 메서드 사용해 불러오세요. 이때, 상품 상세 조회는 인가된 사용자만 이용할 수 있도록 합니다.

  • 상품에 대한 댓글 조회도 가능합니다.

  • 상품 수정 및 삭제 기능을 API를 활용해 구현합니다. 이때, 인가된 사용자만 이용할 수 있도록 합니다.

  • 상품 수정은 '/products/{productId}' PATCH을 사용합니다.

  • 상품 삭제는 '/products/{productId}' DELETE를 사용합니다.

  • 상품 삭제 전, 확인 모달을 띄워주세요.

  • 상품에 대한 좋아요 및 좋아요 취소 기능을 https://panda-market-api.vercel.app/docs에 명세된 '/products/{productId}/favorite' POST & DELETE 활용해 구현합니다. 이때 인가된 사용자만 좋아요 기능을 이용할 수 있도록 합니다.

  • 댓글 생성 및 삭제 기능을 API를 활용해 구현합니다. 이때, 인가된 사용자만 이용할 수 있도록 합니다.

  • 댓글 수정은 https://panda-market-api.vercel.app/docs에 명세된 '/comments/{commentId}' PATCH을 사용합니다.

  • 댓글 삭제는 https://panda-market-api.vercel.app/docs에 명세된 '/comments/{commentId}' DELETE를 사용합니다.
    심화 요구사항
    로그인 및 회원가입 페이지 공통

  • 로그인, 회원가입 기능에 react-hook-form을 활용해 주세요.

  • 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용합니다.
    PC: 1200px 이상
    Tablet: 744px 이상 ~ 1199px 이하
    Mobile: 375px 이상 ~ 743px 이하
    375px 미만 사이즈의 디자인은 고려하지 않습니다
    유저 기능

  • 리퀘스트 헤더에 인증 토큰을 첨부할 때 axios interceptors를 활용해 주세요. (axios를 사용하지 않는다면 이와 유사한 기능을 활용해 주세요.)
    React-Query로 마이그레이션

  • fetch 혹은 axios로 구현된 기존의 API 요청 코드를 React-Qeury로 마이그레이션 합니다.
    로딩 및 에러 핸들링

  • 로딩 인디케이터와 에러 메시지를 구현합니다.

  • 상품 목록 및 상품 상세 데이터를 Prefetching 합니다
    상품 데이터 캐싱 및 업데이트

  • React Query의 캐싱 기능을 활용하여 데이터 로딩 시간을 최소화합니다.

  • 상품 목록 페이지에서 데이터의 실시간 업데이트를 위해 적절한 Query Refresh 설정을 적용합니다.

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@JeonJun02 JeonJun02 requested a review from kimjong95 December 23, 2024 09:33
@JeonJun02 JeonJun02 self-assigned this Dec 23, 2024
@JeonJun02 JeonJun02 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 완성은 아니지만 제출합니다... 제출일 이후 제출한PR 제출 마감일(일요일) 이후에 제출하는 PR입니다. 진행 중 🏃 스프린트 미션 진행중입니다. labels Dec 23, 2024
@JeonJun02 JeonJun02 changed the title sprint9 제출 [전준기] -sprint9 Dec 23, 2024
@JeonJun02 JeonJun02 changed the title [전준기] -sprint9 [전준기] sprint9 Dec 23, 2024
import codeitAxios from "@/lib/codeitAxios";
import { useRouter } from "next/router";

export default function Login() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

토큰기반의 로그인 방식을 잘 정리해주신것 같아요.
로그인이 페이지로 분리가되어있기에 짧은 설명을 추가드려요. 페이지로 분리된 로그인은 유저기 보통 어떠한 기능을 사용할 때 로그인이 필요하면 로그인페이지로 돌려주고 로그인이 완료되면 유저가 사용하던 기능 또는 페이지로 리다이렉트 하는 경우가 많습니다. 따라서 로그인페이지의 URL파라미터로 다음 동작을 설명하는경우가 대부분입니다.

토큰기반 로그인이기 때문에 스토리지에 저장된 토큰이 있다면 로그인페이지에 진입즉시 로그인을 시켜줄 수 있고, 토큰이 만료되었으면 refreshToken기반으로 새로운 인증토큰을 발급받아 유저의 명시적인 로그인 없이도 로그인상태를 유지시켜줄 수 있습니다.
토큰이 아에 없을때 명시적인 로그인을 수행하도록 할 수도 있겠죠. 그러한 플로우를 로그인 페이지에서 고민해보시면 좋을 것 같습니다.

const handleLogout = () => {
if (accessToken) {
localStorage.removeItem("accessToken");
setAccessToken(null);
Copy link
Collaborator

Choose a reason for hiding this comment

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

로그아웃 할 때 accessToken은 삭제해주지만 refreshToken은 삭제하지 않은 이유가 있을까요? refreshToken만 있다면 유저는 로그인을 임의로 수행할 수도 있습니다. 로그아웃이란 유저의 세션을 모두 날리는 작업을 의미합니다. 때문에 특별한 의미가 없다면 refreshToken또한 삭제해주시는게 좋을 수 있습니다.

또한 로그인시에 로컬스토리지에 토큰을 넣는 동작을 수행하는데, 이렇게 로그인시에 토큰을 관리하는 로직을 별도의 훅으로 분리해서 동작시킬 수 있습니다. 그렇게 되면 로컬스토리지에 필요한 키값을 내부의 모듈안에서만 관리할 수 있고, 로그인에 필요한 동작들을 모듈화 할 수 있는 장점이 있습니다.

<Link href="/login" className={styles.topNavLoginBt}>
로그인
</Link>
{accessToken ? (
Copy link
Collaborator

Choose a reason for hiding this comment

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

accessToken이 있는지로 로그인이 확인되면 만로된 accessToken일때는 로그인이 되지않은 상태이지만, 여전히 로그인버튼은 노출되지 않을 수 있습니다.

});
// 성공 시 로컬 스토리지에 토큰 저장장
localStorage.setItem("accessToken", res.data.accessToken);
localStorage.setItem("refreshToken", res.data.refreshToken);
Copy link
Collaborator

Choose a reason for hiding this comment

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

또한 로컬스토리지로 토큰을 관리하는 방식은 Next를 사용한다면 조금 생각해봐야 할 수도 있습니다. 어떠한 페이지를 SSR한다고 할 때 로그인된 유저에게 제공되는 페이지의 정보들이 다를 경우 서버는 유저가 로그인 상태인지 인지할 수 있어야 합니다. 그렇다면 브라우저에서만 사용할 수 있는 localStorage기반의 토큰 인증방식은 서버에서는 사용할 수 없는 스펙이 됩니다. 이러한 경우에는 어떻게 대처할 수 있는지 고민해보아야 합니다.

import Footer from "@/components/Footer";
import Header from "@/components/Header";

export default function Items() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

(사소) items라는 폴더가 존재하니 items/index.jsx로 구성해주시는게 폴더구조 관점에서 더 잘 보일 것 같아요.

Copy link
Collaborator

@kimjong95 kimjong95 left a comment

Choose a reason for hiding this comment

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

전체적으로 Presentational-Container패턴으로 컴포넌트를 잘 구성해주셨고, 코드를 잘 분리해주신 것 같습니다.

로그인은 어떠한 서비스를 만들어도 99%필요한 기능입니다. 로그인 기능을 잘 모듈화 할 수 있다는 점은 100개의 서비스를 만들때 로그인을 고민하지 않아도 될 수 있다는 것이 됩니다.

코드를 분리한다 = 코드를 모듈화한다 라는 개념이 여기서 장점으로 표현될 수 있습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 완성은 아니지만 제출합니다... 제출일 이후 제출한PR 제출 마감일(일요일) 이후에 제출하는 PR입니다. 진행 중 🏃 스프린트 미션 진행중입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants