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

Fix: 탐색 페이지, 리스트 상세 조회 페이지 UI 수정, 로직 수정 #60

Merged
merged 16 commits into from
Feb 23, 2024

Conversation

Nahyun-Kang
Copy link
Contributor

개요

  • 개요는 변경 사항 및 관련 이슈에 대해 간단하게 작성해주세요.
  • 해당 PR에 대한 리뷰어와 라벨을 생성해 주세요.

작업 사항

  • 최상단 page에서 필요없는 params를 삭제
  • 탐색 페이지 콘솔에 에러 안 나오게 수정
  • 탐색 카테고리 컴포넌트 아래 마진 추가 (@kanglocal 참고 부탁드립니다!)
  • description / title 너무 길면 wrapper 넘어가는 문제 수정
  • 리스트 상세 조회 라벨 교체
  • 탐색 헤더 비회원일 때 로그인/로그아웃 클릭 시 로그인 모달이 뜨도록 수정
  • 콜라보레이터 바뀐 api대로 id 수정하여 콜라보레이터 목록에 자신이 제외되도록 수정
  • 답글이 없으면 댓글/답글 작성 불가능하도록 수정

리뷰어에게

  • 추가로 수정 필요한 부분 있으면 언제든지 제보 부탁드립니다!!

@Nahyun-Kang Nahyun-Kang added the Fix 에러/버그 해결 label Feb 23, 2024
@Nahyun-Kang Nahyun-Kang self-assigned this Feb 23, 2024
Copy link

vercel bot commented Feb 23, 2024

@Nahyun-Kang 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.

@Nahyun-Kang Nahyun-Kang changed the title Fix: 탐색 페이지, 리스트 상세 조회 페이지 UI 수정, 로직 수 Fix: 탐색 페이지, 리스트 상세 조회 페이지 UI 수정, 로직 수정 Feb 23, 2024
@Nahyun-Kang Nahyun-Kang merged commit 92d26e3 into 8-Sprinters:dev Feb 23, 2024
1 of 2 checks passed
@Nahyun-Kang Nahyun-Kang deleted the fix/QA branch February 23, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 에러/버그 해결
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant