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

HOTFIX: 리스트생성 imageUrl 미삽입으로 인한 에러 해결 및 기타 수정 #18

Merged

Conversation

seoyoung-min
Copy link
Contributor

@seoyoung-min seoyoung-min commented Feb 6, 2024

개요

  • 리스트생성 리퀘스트에 imageUrl 필드를 빼고 보내, 리스트 조회시 null값으로 인한 에러가 발생하는 현상을 해결했습니다.
    • 해결방법: imageUrl 필드에 빈문자열 넣기
  • 아이템 타이틀 필드 양옆 아이콘이 배포 환경에서 가려지는 에러가 있어 수정했습니다. 로컬과 yarn build-start 환경에서 확인이 불가능하여 함께 올립니다.

작업 사항

  • 에러 해결 (개요 참고)
  • 리스트 생성 시 발생하는 오류 토스트 메시지를 추가했습니다.
  • 리스트 생성 버튼 클릭 시 제출 버튼 비활성화 (클릭후 pending 상태, onSuccess상태에서 비활성)
  • 리스트 생성 리스트 상세 페이지로 이동 (이미지 업로드 성공/실패 모두)
  • 이미지 업로드 retry 3회, 간격 1초로 설정

참고 사항 (optional)

  • 리스트 생성 후 상세 페이지로 리다이렉트 할 때, listId를 이용해 이동해야해서 useState를 사용했습니다. (@ParkSohyunee 도움 주셔서 감사합니다🙇‍♀️)
  • 리다이렉트 링크는 수정된 url인 user/[userId]/list/[listId] 입니다!
  • 아직 토스트메시지 상수화 이전입니다! 참고 부탁드립니다~!
  • 앱라우팅에서 UseRouter의 경로는 import { useRouter } from 'next/navigation' 라는 것을 배웠습니다.

리뷰어에게

  • 이미지 업로드 retry 3회, 간격 1초로 설정 괜찮을지 궁금합니다!
  • 확인 부탁드려요 :)

@seoyoung-min seoyoung-min added Feat 구현 Fix 에러/버그 해결 labels Feb 6, 2024
@seoyoung-min seoyoung-min self-assigned this Feb 6, 2024
Copy link

vercel bot commented Feb 6, 2024

@seoyoung-min is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@Nahyun-Kang Nahyun-Kang left a comment

Choose a reason for hiding this comment

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

에러 해결하시느라 너무 고생많으셨습니다..!! 🥹 LGTM!👍 일단 자세한 내용은 내일 더 꼼꼼히 보겠습니당!

@seoyoung-min seoyoung-min merged commit 1f2ee10 into 8-Sprinters:dev Feb 6, 2024
0 of 2 checks passed
Copy link
Contributor

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

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

서영님 이미지 관련 코드 수정해 주셔서 감사합니다!👍 retry 옵션 설정도 적절한 것 같습니다.
다른 부분도 추가로 확인하도록 하겠습니다!! 감사합니다. 🥰

@seoyoung-min seoyoung-min deleted the feature/create-item-backlog branch June 3, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 구현 Fix 에러/버그 해결
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants