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

Feature/login #56

Merged
merged 3 commits into from
Feb 27, 2023
Merged

Feature/login #56

merged 3 commits into from
Feb 27, 2023

Conversation

Hyeeeein
Copy link
Collaborator

@Hyeeeein Hyeeeein commented Feb 26, 2023

작업 개요 (이슈 번호)

#55

작업 내용 (변경 사항)

  • 설문조사 구현 완료
  • 상세정보 페이지 스타일 수정
  • Loading 컴포넌트 추가 : home 의 상품, 상세 정보 페이지, 추천 상품, 관심 상품
  • /home 과 /detail에 경로 보호 : home 의 경우 refreshToken 이 사라졌을 때 반영이 되지 않아 추가, detail 의 경우 accessToken 이 필요한 상황에 따라 갱신하는 로직 추가
  • useMutation 으로 리팩토링 : 로그인/회원가입

스크린샷

설문조사 구현 완료

image

상세정보 페이지 스타일 수정

image

Loading 컴포넌트 추가

image

/home 과 /detail에 경로 보호

image

useMutation 으로 리팩토링 : 로그인/회원가입

image

테스트 결과

/home 과 /detail에 경로 보호

  • 해당 에러가 콘솔에 찍힘
    image

리뷰 요청 사항

혹시 테스트 결과에 나오는 에러의 해결 방법을 아시는 분 조언 부탁드립니다..

@Hyeeeein Hyeeeein added this to the Develop Completed milestone Feb 26, 2023
@Hyeeeein Hyeeeein self-assigned this Feb 26, 2023
@netlify
Copy link

netlify bot commented Feb 26, 2023

👷 Deploy Preview for tickleplus processing.

Name Link
🔨 Latest commit cecb41b
🔍 Latest deploy log https://app.netlify.com/sites/tickleplus/deploys/63fb933b399c7b00089107ee

@netlify
Copy link

netlify bot commented Feb 26, 2023

Deploy Preview for tickleplus ready!

Name Link
🔨 Latest commit e437a47
🔍 Latest deploy log https://app.netlify.com/sites/tickleplus/deploys/63fbf566bf95c20008e28618
😎 Deploy Preview https://deploy-preview-56--tickleplus.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@Hyeeeein Hyeeeein merged commit abe4677 into develop Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant