Skip to content

[캠퍼스] 필터/글쓰기 버튼 추가 및 변경#1149

Merged
JeongWon-CHO merged 26 commits intofeat/lost-item-sprintfrom
feat/#1148/lostItem-filter
Jan 26, 2026
Merged

[캠퍼스] 필터/글쓰기 버튼 추가 및 변경#1149
JeongWon-CHO merged 26 commits intofeat/lost-item-sprintfrom
feat/#1148/lostItem-filter

Conversation

@JeongWon-CHO
Copy link
Contributor

What is this PR? 🔍

Changes 📝

카테고리가 복수 선택으로 변경되며 type을 LostItemCategory[]로 변경하였습니다.

필터 모달이 추가되었습니다.

  • pc는 모달, 모바일은 바텀시트로 보여줍니다.
    • 바텀시트는 코인오더 프로젝트에서 가져왔습니다.
  • 물품 종류에 한해 복수 선택이 가능합니다.
폴더명 설명
LostItemFilterModal PC에서 띄울 필터 모달
LostItemFilterBottomSheet 모바일에서 띄울 필터 바텀시트
LostItemFilterContent 각 모달에 들어갈 공통 내용

글쓰기 버튼이 변경되었습니다.

  • 변경 전 : 글쓰기 버튼 클릭 시 버튼 기준 좌측에 버튼 생성
  • 변경 후 : 글쓰기 버튼 클릭 시 모달창이 뜸
폴더명 설명
LostItemWriteBottomSheet 모바일에서 띄울 글쓰기 모달

ScreenShot 📷

image image

물품 종류에 한해 복수선택이 가능합니다.

image

비로그인 상태에서 '내 게시글' 필터 선택 시 로그인 모달창이 뜹니다.

image image

모바일에서는 바텀시트 형태로 표시됩니다.

Precaution

세부사항이 디자인과 다릅니다. 감안하고 봐주시면 감사하겠습니다.

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@JeongWon-CHO JeongWon-CHO self-assigned this Jan 26, 2026
@JeongWon-CHO JeongWon-CHO added the 🎓 Campus 학교식단, 버스, 커뮤니티 도메인 label Jan 26, 2026
@github-actions github-actions bot requested review from D0Dam and dooohun January 26, 2026 03:42
Copy link
Contributor

@ff1451 ff1451 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
코멘트 남긴 내용 확인하고 수정 필요하다고 생각되는 부분만 반영해주세요!

@JeongWon-CHO JeongWon-CHO merged commit 0b706f5 into feat/lost-item-sprint Jan 26, 2026
@github-actions github-actions bot deleted the feat/#1148/lostItem-filter branch January 26, 2026 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎓 Campus 학교식단, 버스, 커뮤니티 도메인

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants