Skip to content

Conversation

@b0nsu
Copy link
Collaborator

@b0nsu b0nsu commented Dec 19, 2025

📌 Related Issue Number


Checklist

  • 🎋 base 브랜치를 제대로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (pnpm build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

feat(scrap, folder): implement scrap CRUD, folder management, and enhanced scrap detail view

Features:

  • Implement scrap CRUD operations, search, restore, and trash handling
  • Add folder content screen and trash page with tooltip support
  • Refactor sorting logic and grid layout behavior
  • Improve scrap search and list pages
  • Rename Item components to Card and modularize scrap/trash headers

Scrap Detail Enhancements:

  • Add filter buttons (All / Problem / Pointings) with SegmentedControl
  • Add "View All" button to navigate to AllPointings screen
  • Add expandable modal for problem content (80% screen size)
  • Add maximize button on hover for problem content
  • Convert pointings to accordion format for better UX

API & Type Safety:

  • Implement ScrapDetailResp → PublishProblemGroupResp data transformation
  • Convert PointingResp to PointingWithFeedbackResp format
  • Convert ProblemExtendResp to ProblemWithStudyInfoResp format
  • Add type-safe data mapping using schema.d.ts types
  • Handle missing fields with appropriate defaults for AllPointings navigation

💡 New Insights & Learnings


📢 To Reviewers

  • *** This is a DEMO version for backup purposes; UI and detailed features will be updated later.

📸 Screenshot or Video (Optional)

2025-12-19.11.36.48.mov

@b0nsu b0nsu changed the title Feat/native/scrap #138 2 [feat/native] 스크랩 페이지 구현 초기 PR #148 Dec 19, 2025
@b0nsu b0nsu requested a review from sterdsterd December 19, 2025 14:40
@b0nsu b0nsu changed the base branch from main to native December 19, 2025 14:41
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.

2 participants