Skip to content

feature: 사이드바에서 멤버 탭 제거 - #220

Merged
minbros merged 13 commits into
devfrom
feature/sidetab-member-detach
Oct 10, 2026
Merged

minbros merged 13 commits into
devfrom
feature/sidetab-member-detach

Conversation

@KangShinGyu98

@KangShinGyu98 KangShinGyu98 commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

변경 내용

  • 사이드텝에 멤버 초대하기를 제거하고 상단 헤더에 통합함
  • 상단 헤더에서 멤버/초대하기 분리하고 드롭다운 + 모달로 변경함
  • 초대링크 : 방장만 볼 수 있어요. 수정 필요

변경 이유

  • 멤버 초대 위계 정립, 사이드텝 정리, 초대하기로 워딩 통일

테스트

  • npm run lint
  • npm test
  • npm run build
  • 그 외 수동 검증: 모바일, pc크롬, pc사파리, 사파리 아이폰 에뮬 확인

“KangShinGyu” and others added 8 commits October 6, 2026 22:54
- Figma 원본(24×24) copy, copy-check, refresh, user-edit, user-x, menu-dot-horizontal 추가
- Figma 미등록 종이비행기는 Tabler send(outline)를 stroke 1.8로 임시 추가
- icons/index.tsx에 export 추가

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- UserAvatar: 배경·이니셜 스타일을 바꿀 className, initialClassName 추가
- SidebarPingBadge: 위치를 덮어쓸 className 추가
- SettingsDialog: medium(600px) 크기, className 병합, 제목 옆 titleAccessory, data-skip-autofocus 첫 포커스 제외 추가
- ConfirmDialog: ledger 설명의 줄바꿈(\n) 표시
- DestinationSearchInput: 48px 입력칸 모양 appearance="field" 추가 (기본 동작 유지)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 위임 성공 시 방 목록·방 상세·구성원 캐시의 역할을 바로 갱신
- 방 상세 캐시도 재조회 대상에 추가
- 위임 후 나가기 시 "먼저 방장을 위임해 주세요"가 다시 뜨던 문제 수정

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 확인 절차는 유지하고 ConfirmDialog(ledger) 기반으로 Figma 문구·버튼 적용
- DeleteConfirmModal이 id·title만 받도록 타입 완화

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 헤더 우측에 온라인 구성원 최대 3명 아바타와 나머지 +N, 초대하기 링크 추가
- 참여 요청이 있으면 아바타 우상단에 깜빡임 배지 표시
- 사이드바 멤버 탭 제거, 북마크·가계부 사이 구분선 추가
- 모바일 메뉴의 멤버 관리를 초대하기로 변경
- 관련 테스트 기대값 갱신

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 초대 링크·구성원을 항상 표시하고 닫기·멤버 초대 토글 제거
- 링크 칸 안에 복사·공유 버튼 배치, 재발급·자동 발급은 방장만
- 참여 요청·구성원 카드를 Figma 디자인으로 변경, 옵션 메뉴(방장 위임·강제 퇴장) 정리
- 나가기·위임·강제 퇴장 확인을 팝업으로 통일, 여행 삭제는 여행 정보 수정으로 이동
- 다른 방 요청을 띄우던 MemberSettingsHostGuard 제거

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 수정 모달 전용 입력 필드(RoomTripEditFields) 분리, 새 여행 만들기 필드는 유지
- 라벨을 칸 밖으로, 48px 입력칸·시작일/종료일 분리·날짜 표기 형식 적용
- 취소·적용하기 버튼 스타일 정리, 모달 너비 600px
- 방장 전용 여행 삭제를 제목 옆에 배치

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Figma 승인 대기 카드(상태 칩·제목·방 이름·안내·승인 상태 확인) 적용
- 거절 상태 문구와 홈으로 돌아가기 버튼 정리

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
frontend-server Ready Ready Preview Oct 10, 2026 8:04am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 16 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 39490f85-1076-4480-a9bf-2c14675876e5

📥 Commits

Reviewing files that changed from the base of the PR and between f409d41 and abdc3f6.


⛔ Files ignored due to path filters (7)
  • src/assets/icons/copy-check.svg is excluded by !**/*.svg
  • src/assets/icons/copy.svg is excluded by !**/*.svg
  • src/assets/icons/menu-dot-horizontal.svg is excluded by !**/*.svg
  • src/assets/icons/refresh.svg is excluded by !**/*.svg
  • src/assets/icons/send.svg is excluded by !**/*.svg
  • src/assets/icons/user-edit.svg is excluded by !**/*.svg
  • src/assets/icons/user-x.svg is excluded by !**/*.svg

📒 Files selected for processing (43)
  • src/app/(main)/member-settings/_components/AddMemberPanel.tsx
  • src/app/(main)/member-settings/_components/JoinRequestsSection.tsx
  • src/app/(main)/member-settings/_components/MemberCard.tsx
  • src/app/(main)/member-settings/_components/MemberList.tsx
  • src/app/(main)/member-settings/_components/MemberSettingsHostGuard.tsx
  • src/app/(main)/member-settings/_components/RoomMembersSection.tsx
  • src/app/(main)/member-settings/_components/useMemberActions.tsx
  • src/app/(main)/room-settings/_components/RoomTripSettingsSection.tsx
  • src/app/home/_components/DeleteConfirmModal.tsx
  • src/app/home/_components/InviteRoomModal.tsx
  • src/app/home/_components/LeaveConfirmModal.tsx
  • src/app/home/_components/RoomCardMenu.test.tsx
  • src/app/waiting/page.tsx
  • src/assets/icons/index.tsx
  • src/components/analytics/CookieSettingsEntrypoints.test.tsx
  • src/components/layout/HeaderBar.tsx
  • src/components/layout/HeaderMemberPreview.tsx
  • src/components/layout/SideBar.tsx
  • src/components/layout/SidebarPingBadge.tsx
  • src/components/layout/mobile-header.test.tsx
  • src/components/layout/planning-chrome.test.tsx
  • src/components/layout/travel-navigation.test.tsx
  • src/components/map/useMobileMapTextSearch.ts
  • src/components/rooms/RoomTripEditDialog.test.tsx
  • src/components/rooms/RoomTripEditDialog.tsx
  • src/components/rooms/RoomTripEditFields.tsx
  • src/components/rooms/RoomTripEditForm.test.tsx
  • src/components/rooms/RoomTripEditForm.tsx
  • src/components/search/DestinationSearchInput.tsx
  • src/components/settings/ConfirmDialog.tsx
  • src/components/settings/SettingsActionButton.tsx
  • src/components/settings/SettingsDialog.tsx
  • src/components/user/UserAvatar.tsx
  • src/contexts/stomp/StompProvider.tsx
  • src/hooks/useRooms.ts
  • src/lib/rooms/host-transfer-cache.test.ts
  • src/lib/rooms/host-transfer-cache.ts
  • src/lib/rooms/member-preview.test.ts
  • src/lib/rooms/member-preview.ts
  • src/lib/stomp/forced-room-exit-dispatch.ts
  • src/lib/stomp/join-requests-dispatch.ts
  • src/lib/stomp/subscribe-room-topics.test.ts
  • src/lib/stomp/subscribe-room-topics.ts

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

- 기존 초대 링크를 재사용하고 재발급·삭제 권한은 방장으로 유지

- 멤버 권한 회귀 테스트와 날짜 선택 키보드 동작 보강
- PR #220에서 잘못 포함된 멤버 권한 확대 변경을 되돌림
- 날짜 선택 입력의 키보드 접근성 보완 및 회귀 테스트 추가
- 중첩 조건과 표시 로직을 분리하고 읽기 전용 props 적용
@minbros minbros changed the title Feature/sidetab member detach feature: 사이드바에서 멤버 탭 제거 Oct 10, 2026
- 채팅 라우팅과 헤더 구성원 관리 변경을 함께 유지
- 양쪽 아이콘과 내비게이션 회귀 테스트 통합
@sonarqubecloud

Copy link
Copy Markdown

@minbros
minbros merged commit 0f54c5e into dev Oct 10, 2026
4 checks passed
@minbros
minbros deleted the feature/sidetab-member-detach branch October 10, 2026 08:07

This branch was successfully deployed

1 active deployment
Preview — abdc3f64 Deployed Oct 10, 2026 by vercel[bot]
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