Skip to content

fix: 매장운영 워크스페이스 카드 진행률 제거 및 카드 높이 통일 (#80) - #82

Merged
wjswlgh96 merged 1 commit into
developfrom
fix/#80/workspace-card-purpose-metric
Jul 21, 2026
Merged

fix: 매장운영 워크스페이스 카드 진행률 제거 및 카드 높이 통일 (#80)#82
wjswlgh96 merged 1 commit into
developfrom
fix/#80/workspace-card-purpose-metric

Conversation

@wjswlgh96

@wjswlgh96 wjswlgh96 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Pull Request

작업 내용

  • 워크스페이스 목록 카드에서 매장운영(store-operation) 템플릿은 진행률 지표를 표시하지 않도록 수정
  • 진행률 유무와 무관하게 모든 카드 높이를 통일

작업 결과

  • 매장운영 카드: 진행률 바·% + task 완료 카운트(task 기반 지표) 미표시 → 아이콘·이름·용도·멤버수·최근활동만 표시
  • 팀/사이드 프로젝트 카드: 기존 진행률·완료 카운트 그대로 유지
  • 진행률이 빠진 매장운영 카드도 다른 카드와 동일한 높이(141px)로 정렬

작업 배경

  • get_my_workspaces RPC가 purpose와 무관하게 tasks 테이블만 집계해 progress = 완료 task / 전체 task를 내려줌
  • 매장운영 템플릿은 task 보드 모듈이 없어(활성 모듈: work_schedule) 항상 0/0 완료 · 0%로 표시 → 개념적으로 무의미하고 에러처럼 보임
  • 팀 회의 결정: 매장운영 UI에서는 진행률 개념을 제거 (근무 지표로 대체하지 않음)

변경 사항

Fixed

  • 매장운영 워크스페이스 카드의 무의미한 진행률(0/0 · 0%) 표시 제거
  • 진행률 제거로 매장운영 카드만 높이가 짧아지던 문제 → min-h로 카드 높이 통일

Added / Changed

  • (없음 — RPC/타입 변경 없이 카드 UI 조건 분기만 수정)

상세

  • WorkspaceCardshowTaskProgress = purpose !== 'store-operation' 플래그 추가
  • 진행률 바 + "N/N 완료" 카운트(둘 다 task 기반 지표)를 showTaskProgress로 조건부 렌더
  • 카드 Linkmin-h-[141px](진행률 있는 카드의 자연 높이) 적용해 목록 카드 높이 통일
  • RPC 변경 없음 — 서버는 기존대로 반환하고, 매장운영일 때 프론트에서 해당 지표만 미표시

실행화면

  • 매장운영(카페 브라운)은 진행률 미표시, 팀/사이드는 유지, 카드 높이 동일

테스트

  • 로컬 실행 확인 (test1 로그인 후 목록 렌더 확인)
  • 주요 시나리오 확인 (매장운영 진행률 미표시 / 팀·사이드 진행률 유지 / 전 카드 높이 141px 실측)
  • 영향 범위 확인 (WorkspaceCard 단일 파일, RPC·타입 무변경, npm run check 통과)

리뷰 체크리스트

  • PR base branch가 올바릅니다. (feature/* -> develop, 배포 시 develop 또는 release/* -> main)
  • 브랜치명이 Type/#issue-number/description 형식을 따릅니다.
  • 커밋 메시지가 컨벤션을 따릅니다.
  • 불필요한 console.log, 주석, 임시 코드를 제거했습니다.
  • 타입 에러와 린트 에러를 확인했습니다.
  • CodeRabbit 1차 리뷰를 확인했습니다.
  • CodeRabbit 리뷰 반영 후 Discord에 공유했습니다.
  • 최소 1명 이상의 approve 후 merge합니다.

리뷰 요청사항

  • 매장운영 카드에서 진행률 바뿐 아니라 "N/N 완료" 카운트도 함께 제거했습니다(둘 다 task 기반이라 완료 카운트만 남으면 0/0으로 여전히 어색). 완료 카운트는 남기는 게 낫다는 의견 있으면 알려주세요.
  • 카드 높이 통일은 min-h-[141px](진행률 있는 카드의 자연 높이)로 처리했습니다.

관련 이슈

Closes #80

Summary by CodeRabbit

  • 개선 사항
    • 매장 운영 목적의 워크스페이스 카드에서 작업 완료 수와 진행률 표시를 숨깁니다.
    • 작업 진행 정보가 없는 경우에도 카드 높이가 일정하게 유지되어 목록 화면이 더 일관되게 표시됩니다.

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
syncly Ready Ready Preview, Comment Jul 21, 2026 12:50am

@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

WorkspaceCard가 워크스페이스 목적에 따라 task 진행률 UI를 표시하도록 변경되었습니다. 매장운영 워크스페이스에서는 완료 수와 진행률 바를 숨기며, 카드 최소 높이를 고정합니다.

Changes

워크스페이스 카드 진행률 표시 제어

Layer / File(s) Summary
목적별 진행률 렌더링
src/widgets/workspace-list/ui/WorkspaceCard.tsx
store-operation이 아닌 워크스페이스에만 완료 task 수와 진행률 바를 렌더링하고, 모든 카드에 최소 높이를 적용합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • TeampleRun/syncly#10: WorkspaceCard에 진행률 및 완료 수 UI를 추가한 변경으로, 현재 조건부 숨김 로직과 직접 연결됩니다.

Suggested reviewers: kwon812

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed 이슈 #80의 진행률·완료 카운트 제거, 높이 통일, RPC 미변경 요구를 충족합니다.
Out of Scope Changes check ✅ Passed WorkspaceCard UI 조건 분기와 카드 높이 통일만 포함되어 있어 범위를 벗어난 변경은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed 주요 변경인 매장운영 워크스페이스 카드의 진행률 제거와 높이 통일을 간결하게 잘 요약합니다.
Description check ✅ Passed 필수 섹션과 핵심 정보가 대부분 채워져 있으며, 작업 내용·결과·테스트·이슈까지 목적에 맞게 정리돼 있습니다.
✨ 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 fix/#80/workspace-card-purpose-metric

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.

@seongjinss555 seongjinss555 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

고생하셨습니다~

@wjswlgh96
wjswlgh96 merged commit f258637 into develop Jul 21, 2026
3 checks passed
@wjswlgh96
wjswlgh96 deleted the fix/#80/workspace-card-purpose-metric branch July 21, 2026 01:21
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.

fix: 매장운영 워크스페이스 카드에서 진행률 지표 제거

2 participants