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

Feat: 요청주제 페이지 개발 #257

Merged
merged 23 commits into from
Oct 27, 2024

Conversation

Eugene-A-01
Copy link
Contributor

개요

  • 요청주제 (/topics)

작업 사항

요청주제 메인페이지(/topics)

  • 무한스크롤
  • 하단 그라데이션
  • 요청주제 클릭시 리스트 템플릿 생성으로 바로 이동 (/list/create?title=제목&category=카테고리)
  • 주제 요청하기 클릭시 바텀시트
  • 비회원 요청주제 클릭시 로그인 모달
  • 비회원 주제 요청하기 버튼 클릭시 로그인 모달

바텀시트

  • 리스트 생성 정책에 따라, 주제는 30자 제한 (어길 시 에러메시지)
  • 요청 보내기 버튼은 주제가 1자 이상 30자 이하일 때 활성화 됨.
  • 카테고리 드롭다운 (기본은 전체)
  • 익명 토글버튼 (기본 실명)
  • 요청 보내기 API 에러시 에러메시지
  • 요청 보내기 성공시 모달

참고 사항

  • 하루 최대 3번까지 주제 요청 정책은 백로그로 구현하겠습니다!
  • API 연결 코드는 구현해놓았고, API가 안나와서 아직 테스트를 못 해봤습니다. 최고멋진 동호님 작업 끝나시는대로 연결 잘 되는지 확인하고 머지 하겠습니다~! (현재는 mockdata로 주제목록 그려지고 있음)

스크린샷

  1. 요청주제목록 페이지 (/topics)
image
  1. 주제 요청하기 클릭
    (1) 로그인 상태에서는 바텀시트
image image
(2) 비회원 접근시 로그인 모달
image
  1. 요청 보내기 클릭
    (1) 주제 요청 성공시 성공 모달
image
(2) 주제 요청 실패시(API 실패) 에러메시지
image

리뷰어에게

  • 바텀시트 디자인을 알아서 한 부분이 있어서, 눈에 거슬리는 부분이 있다면 수정 조언 부탁드립니다. :)

Copy link

vercel bot commented Oct 6, 2024

@Eugene-A-01 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

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

유진님~ 요청 주제 기능 빨리 사용해보고 싶어요! 감사합니다.💕 바텀 시트 디자인도 찰떡입니다!! 🎨

Comment on lines +46 to +60
<button
className={styles.goBackButton}
onClick={() => {
router.back();
}}
>
뒤로가기
Copy link
Contributor

Choose a reason for hiding this comment

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

유진님~ 사소하지만 해당 부분은 나중에 헤더 공통 컴포넌트 작업 완료후에 변경하면 좋을 것 같아서 코멘트 남깁니다~!! 👍

@Eugene-A-01 Eugene-A-01 merged commit 29031a8 into 8-Sprinters:dev Oct 27, 2024
1 of 2 checks passed
Copy link
Contributor Author

@Eugene-A-01 Eugene-A-01 Oct 30, 2024

Choose a reason for hiding this comment

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

@ParkSohyunee
모달 버튼이 안눌리는것 때문에 이 코드 작성해서 해결했는데,
모달 닫히는게 혹시 이것 때문일까요?

최신 pull 받지말라고 하셔서 로그인모달 바로 닫히는 이슈 확인 못해보고 있어서 한 번 이 라인 지우고 확인 부탁드립니다!!

pointerEvents: 'none',

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants