Skip to content

[Refactor] 이슈보드를 작업보드로 변경#36

Merged
yakjun01 merged 4 commits into
mainfrom
refactor/28-move-issue-to-work
May 29, 2026
Merged

[Refactor] 이슈보드를 작업보드로 변경#36
yakjun01 merged 4 commits into
mainfrom
refactor/28-move-issue-to-work

Conversation

@yakjun01
Copy link
Copy Markdown
Collaborator

@yakjun01 yakjun01 commented May 29, 2026

🔎 What

  • nav 에서 이슈 보드 삭제
  • 워크스페이스에서 작업보드를 이슈보드 UI 및 데이터로 변경

🔗 Issue

✅ 체크리스트

  • 브랜치 base가 적절한가요?
  • 제목이 이슈 제목과 동일한가요?
  • 최소 1명의 리뷰를 받았나요?

@yakjun01 yakjun01 force-pushed the refactor/28-move-issue-to-work branch from 08b1a74 to 60b2d0d Compare May 29, 2026 00:49
Copy link
Copy Markdown
Collaborator

@slyhyun slyhyun left a comment

Choose a reason for hiding this comment

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

ChatPage.tsx (line 1572) 워크스페이스별로 사이드바는 visibleRepositories를 쓰는데, OverviewPanel에는 전체 repositories를 넘기고 있음.
그래서 워크스페이스를 바꿔도 통합 개요의 repo 수/PR/이슈 합계와 카드 목록은 전체 워크스페이스 기준으로 보일 수 있습니다. repositories={visibleRepositories}가 맞아 보임.

추가로 ChatPage.tsx (line 68) 여러 repo가 같은 채널 id(frontend-chat, backend-chat, review-room)를 공유하는데, reverse map은 마지막 repo만 남김.
그래서 ChannelPanel 전달 (line 1590) 시 선택한 워크스페이스의 repo가 아니라 *-3 repo로 고정될 수 있음.
여기서는 REPO_CHANNEL_IDS_REVERSE[selectedChannel]보다 selectedRepository를 쓰는 쪽이 안전해보임.

Copy link
Copy Markdown
Collaborator

@slyhyun slyhyun left a comment

Choose a reason for hiding this comment

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

lgtm!!

@yakjun01 yakjun01 force-pushed the refactor/28-move-issue-to-work branch from fe5d949 to ae4aba4 Compare May 29, 2026 01:47
@yakjun01 yakjun01 merged commit 8c5723e into main May 29, 2026
1 check passed
@yakjun01 yakjun01 deleted the refactor/28-move-issue-to-work branch May 29, 2026 01:53
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.

[Refactor] 이슈보드를 작업보드로 변경

2 participants