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

[Dev-Log] 박진현 2021-12-13 #440

Open
developerjhp opened this issue Dec 15, 2021 · 0 comments
Open

[Dev-Log] 박진현 2021-12-13 #440

developerjhp opened this issue Dec 15, 2021 · 0 comments
Assignees
Labels
dev-log dev 로그 작성시 사용하는 라벨

Comments

@developerjhp
Copy link
Collaborator

오늘은 어떻게 프로젝트에 기여했나요?

  • 헤더 현재페이지 볼드처리
  • 어드민페이지 코드,css 수정
  • [서버]product컨트롤러 수정
  • Users 타입 수정
  • 랜딩페이지 회원가입버튼 로그인시 마이페이지로변경
  • 모든 axios요청실패시 에러페이지로 리다이렉트
  • 토큰만료시 로그인페이지로 리다이렉트
  • 홈 작가보러가기 버튼 링킹안되는 문제 해결 및 cursor pointer 추가

오늘의 프로젝트에서 힘든 점은 무엇인가요?

  • 토큰만료시 로그인페이지로 리다이렉트
      .catch((err) => {
      if (err.response.status === 401 && isLogin) {
         alert('로그인이 만료되었습니다. 다시 로그인해주세요.');
         localStorage.removeItem('lumiereUserInfo');
         setIsLogin(false);
         window.location.assign('/signin');
        }
   });

처음엔 err.response.status === 401일 경우에만 로그인페이지로 보냈었는데, 그렇게 하면 로그인이필요한 서비스를
비로그인 유저가 누를경우 나타나는 "로그인이 필요한 서비스입니다. 로그인을 해주세요" 모달이 나오는 대신 '로그인이 만료되었습니다.' 경고창이
뜬다. 그래서, 401이면서도 로그인이 된 유저를 대상으로 걸러내야한다

오늘의 회고

오늘은 계속해서 에러핸들링을 담당하고 있어서 머리가 상당히 아팠다
그래도 에러가 해결될때의 쾌감은 최고^^!!

@developerjhp developerjhp added the dev-log dev 로그 작성시 사용하는 라벨 label Dec 15, 2021
@developerjhp developerjhp self-assigned this Dec 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-log dev 로그 작성시 사용하는 라벨
Projects
None yet
Development

No branches or pull requests

1 participant