Skip to content

Add community chips#555

Open
yoonc01 wants to merge 2 commits into
mainfrom
add-community-chips
Open

Add community chips#555
yoonc01 wants to merge 2 commits into
mainfrom
add-community-chips

Conversation

@yoonc01

@yoonc01 yoonc01 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

작업 내용

  • 커뮤니티에 동행, 중고거래 항목을 추가했습니다.
  • university-web에서 불필요한 커뮤니티 관련 코드를 제거했습니다.

@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-university-web Ready Ready Preview, Comment Jun 14, 2026 1:40pm
solid-connection-web Ready Ready Preview, Comment Jun 14, 2026 1:40pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jun 14, 2026 1:40pm

@yoonc01 yoonc01 self-assigned this Jun 14, 2026
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b6d999cf-b8ad-432b-9c7c-cc32d3967e8b

📥 Commits

Reviewing files that changed from the base of the PR and between dfe8930 and 50fd185.

📒 Files selected for processing (21)
  • apps/university-web/src/apis/community/api.ts
  • apps/university-web/src/apis/community/deleteComment.ts
  • apps/university-web/src/apis/community/deleteLikePost.ts
  • apps/university-web/src/apis/community/deletePost.ts
  • apps/university-web/src/apis/community/getBoard.ts
  • apps/university-web/src/apis/community/getBoardList.ts
  • apps/university-web/src/apis/community/getPostDetail.ts
  • apps/university-web/src/apis/community/getPostList.ts
  • apps/university-web/src/apis/community/index.ts
  • apps/university-web/src/apis/community/patchUpdateComment.ts
  • apps/university-web/src/apis/community/patchUpdatePost.ts
  • apps/university-web/src/apis/community/postCreateComment.ts
  • apps/university-web/src/apis/community/postCreatePost.ts
  • apps/university-web/src/apis/community/postLikePost.ts
  • apps/university-web/src/apis/community/server.ts
  • apps/university-web/src/constants/community.ts
  • apps/university-web/src/types/community.ts
  • apps/web/src/app/community/[boardCode]/[postId]/modify/PostModifyForm.tsx
  • apps/web/src/app/community/[boardCode]/create/PostForm.tsx
  • apps/web/src/app/community/_components/PostCategorySelector.tsx
  • apps/web/src/constants/community.ts
💤 Files with no reviewable changes (17)
  • apps/university-web/src/apis/community/getBoard.ts
  • apps/university-web/src/apis/community/deletePost.ts
  • apps/university-web/src/apis/community/getBoardList.ts
  • apps/university-web/src/apis/community/getPostList.ts
  • apps/university-web/src/apis/community/postLikePost.ts
  • apps/university-web/src/types/community.ts
  • apps/university-web/src/apis/community/patchUpdateComment.ts
  • apps/university-web/src/apis/community/getPostDetail.ts
  • apps/university-web/src/apis/community/postCreateComment.ts
  • apps/university-web/src/apis/community/server.ts
  • apps/university-web/src/apis/community/api.ts
  • apps/university-web/src/apis/community/postCreatePost.ts
  • apps/university-web/src/constants/community.ts
  • apps/university-web/src/apis/community/deleteComment.ts
  • apps/university-web/src/apis/community/deleteLikePost.ts
  • apps/university-web/src/apis/community/index.ts
  • apps/university-web/src/apis/community/patchUpdatePost.ts

Walkthrough

이번 PR은 두 가지 독립적인 변경을 포함합니다.

  1. apps/university-web 커뮤니티 모듈 전체 삭제

    • types/community.ts의 도메인 인터페이스(Board, Post, Comment 등 12종) 제거
    • constants/community.tsCOMMUNITY_BOARDS, COMMUNITY_CATEGORIES, COMMUNITY_MAX_UPLOAD_IMAGES 상수 삭제
    • apis/community/api.tscommunityApi 객체 및 쿼리 키, 응답 타입 인터페이스 전체 제거
    • server.tsgetPostListServer 서버 함수 및 React Query 기반 훅 11종(useDeleteComment, useDeletePost, useGetBoard, useGetPostList 등) 삭제
    • index.ts 배럴 재export 전체 제거
  2. apps/web 게시글 분류 선택 UI 리팩터링

    • constants/community.tsCOMMUNITY_POST_CATEGORIES 추가 및 COMMUNITY_CATEGORIES 5개 항목으로 확장
    • PostCategorySelector 클라이언트 컴포넌트 신규 추가
    • PostFormPostModifyForm에서 isQuestion 토글을 postCategory 상태 기반으로 전환하고 UI를 PostCategorySelector로 교체

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20분

Suggested reviewers

  • wibaek
  • enunsnv
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명에서 작업 내용은 간략히 언급되었으나, 관련 이슈, 특이 사항, 리뷰 요구사항 등 템플릿의 필수 섹션이 누락되었습니다. 관련 이슈 번호, 기술적 특이 사항(커뮤니티 모듈 제거로 인한 영향범위), 리뷰 중점 사항을 추가하여 템플릿을 완성해주세요.
Title check ❓ Inconclusive 제목이 '동행'과 '중고거래' 카테고리 추가라는 주요 변경사항을 반영하지만, 대규모 코드 제거 작업도 포함되어 있어 제목이 전체 범위를 완전히 나타내지 못합니다. 제목을 'Add community chips and remove legacy university-web community code' 또는 'Add community categories (carpooling, secondhand) and clean up legacy code'로 구체화하여 주요 변경사항을 모두 반영하는 것을 권장합니다.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-community-chips

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the web label Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant