Skip to content

[Fix] 컬렉션 삭제 확인창의 문서 삭제 안내 수정 - #445

Merged
Gimini-3 merged 1 commit into
developfrom
fix/444
Oct 7, 2026
Merged

Gimini-3 merged 1 commit into
developfrom
fix/444

Conversation

@Gimini-3

@Gimini-3 Gimini-3 commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Closes #444

문제

컬렉션 삭제 확인창이 "하위 컬렉션과 그 안의 문서도 전부 함께 삭제됩니다"라고 안내했지만, 삭제 API(CollectionCommandService.deleteCollection)는 컬렉션과 하위 컬렉션을 soft delete하고 문서 연결(collection_documents)·컬렉션 권한·closure 행만 지웁니다. 문서 자체는 상태와 deleted_at이 바뀌지 않아 문서 목록에 그대로 남습니다.

기능명세서 시험항목(2. 컬렉션 삭제)의 기대 결과도 "문서는 삭제되지 않고 문서 목록에 그대로 남는다"여서, 시험 중 확인창 안내와 실제 결과가 어긋났습니다.

변경

  • frontend/app/features/CollectionsPage.tsx 확인창 문구를 실제 동작에 맞췄습니다.
    • 변경 전: 하위 컬렉션과 그 안의 문서도 전부 함께 삭제됩니다.
    • 변경 후: 하위 컬렉션도 함께 삭제되며, 들어 있던 문서는 컬렉션에서만 빠지고 문서 자체는 유지됩니다.
  • 복구 API가 없다는 안내는 그대로 둡니다. 문서 연결과 권한은 실제로 삭제되어 컬렉션 상태만 되돌려서는 복구되지 않습니다.

검증

시험 결과
npm run lint 통과
npm test (빌드 포함) 49/49 통과

브라우저에서 확인창을 직접 띄워 확인하지는 않았습니다.

범위와 한계

  • 컬렉션 삭제 API 동작은 바꾸지 않았습니다.
  • 소유자가 아닌 사용자에게도 '컬렉션 삭제' 버튼이 보이는 문제(백엔드가 PERMISSION_DENIED로 거부)는 이번 범위가 아닙니다.
  • 설계 문서 docs/design/kangcheolung-#229-collection-tree.md의 옛 문구 인용은 당시 설계 기록이라 그대로 둡니다.

@Gimini-3 Gimini-3 added the 🐞 Fix 버그 수정 label Oct 7, 2026
@Gimini-3 Gimini-3 self-assigned this Oct 7, 2026
@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: DocGrid/docgrid/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 884f0e06-9775-4267-8f68-c1e10c196e99
  • Autopilot · 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.

@Gimini-3
Gimini-3 merged commit 6e27543 into develop Oct 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 컬렉션 삭제 확인창이 문서도 삭제된다고 잘못 안내

1 participant