Skip to content

fix: 프로젝트 파일 추가 1개 제한 및 대기 파일 표시 개선 (#405) - #406

Merged
kcleverp merged 2 commits into
devfrom
fix/kcleverp-project-file-single-add
Aug 12, 2026
Merged

fix: 프로젝트 파일 추가 1개 제한 및 대기 파일 표시 개선 (#405)#406
kcleverp merged 2 commits into
devfrom
fix/kcleverp-project-file-single-add

Conversation

@kcleverp

Copy link
Copy Markdown
Collaborator

이슈

Closes #405

변경 사항

  • ProjectFileUploadModal에서 한 번에 파일 하나만 추가하도록 제한 — 파일마다 이름·설명·즐겨찾기를 개별로 지정하는 구조인데, 여러 파일을 동시에 받으면 그 값들을 어느 파일에 적용해야 할지 애매했던 문제 해결
  • 파일을 추가하면 드롭존이 파일명·최종 수정일·업로드한 사람이 보이는 행 하나로 바뀜 (최종 수정일·업로드한 사람은 아직 서버에 없는 값이라 추가 시각/현재 로그인 사용자로 미리보기)
  • 즐겨찾기는 행의 별 아이콘으로 토글, 제거는 ⋮ 더보기 메뉴로 통일 (기존 "제거" 텍스트 버튼 대체)
  • 파일 제거 시 드롭존으로 복귀

스크린샷

리뷰 포인트

  • 파일 1개 제한이 실제 사용 흐름에서 불편하지 않은지
  • 최종 수정일/업로드 컬럼을 저장 전 미리보기 값으로 보여주는 게 헷갈리지 않는지

체크리스트

  • 셀프 리뷰 완료
  • 동작 확인 (Prettier/ESLint/tsc -b/vite build 통과 + 브라우저에서 파일 추가/제거/즐겨찾기 토글 직접 확인)

- 한 번에 파일 하나만 추가하도록 제한 (여러 파일일 때 이름/설명/즐겨찾기 적용 대상이 애매하던 문제 해결)
- 파일을 추가하면 드롭존 대신 파일명·최종 수정일·업로드한 사람이 보이는 행으로 표시
- 즐겨찾기는 별 아이콘으로, 제거는 ⋮ 메뉴로 통일 (기존 텍스트 버튼 대체)
- 파일 제거 시 드롭존으로 복귀
@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
slate-to-fe Ready Ready Preview Aug 12, 2026 12:17pm

@kcleverp

Copy link
Copy Markdown
Collaborator Author

코드 리뷰 (버그/안정성 중심)

Must

1. 파일 제거 후 다른 파일을 추가하면 이전 파일명이 그대로 남음

removeFilequeuedFilenull로 지우고 fileName/description은 그대로 둡니다.

  1. fileA.png 드롭 → fileName이 "fileA.png"로 자동 채워짐
  2. ⋮ 메뉴로 fileA.png 제거
  3. fileB.pdf 드롭 → addFilesif (!fileName.trim()) 체크가 false라 자동 채움이 안 되고, fileName은 여전히 "fileA.png"
  4. 저장 → 실제로는 fileB.pdf 내용인데 프로젝트 파일 목록에는 "fileA.png"라는 이름으로 저장됨 (확장자까지 어긋남)

removeFile에서 fileName/description도 같이 초기화하거나, 최소한 fileName을 지워서 다음 파일 추가 시 자동 채움이 다시 동작하게 해야 할 것 같습니다.

Should

2. 드래그로 여러 파일을 한 번에 놓으면 나머지가 조용히 버려짐

FileInputmultiple을 안 넘겨서 기본값 false가 되는데, 이건 네이티브 파일 선택 다이얼로그의 다중 선택만 막을 뿐 드래그앤드롭 자체는 여전히 여러 파일을 받을 수 있습니다. FileInput.selectFiles!multiple이면 조용히 validFiles[0]만 남기기 때문에, 사용자가 3개를 드롭해도 안내 없이 1개만 추가됩니다. "한 번에 하나만 추가할 수 있습니다" 같은 안내를 hint나 에러 메시지에 추가하면 좋을 것 같습니다.


버그·안정성 관점 위주로만 봤습니다. Must 1건은 반영하겠습니다.

- removeFile에서 fileName/description도 같이 초기화 (제거 후 다른 파일 추가 시 이전 파일명이 남아 잘못 저장되던 문제)
- FileInput에 onExtraFilesIgnored 콜백 추가 — multiple=false인데 여러 파일을 드롭하면 안내 메시지 표시 (기존엔 나머지가 안내 없이 조용히 버려짐)
@kcleverp

Copy link
Copy Markdown
Collaborator Author

Must/Should 둘 다 반영 완료했습니다.

  • removeFile에서 fileName/description도 같이 초기화 — 제거 후 다른 파일 추가해도 더 이상 이전 파일명이 안 남음
  • FileInput에 onExtraFilesIgnored 콜백 추가, 여러 파일 드롭 시 "파일은 한 번에 하나만 추가할 수 있어요" 안내 표시. onChange보다 뒤에 호출되도록 순서를 맞춰서 addFiles의 setError('')에 안내 메시지가 바로 덮어써지지 않게 했습니다 (다른 FileInput 사용처인 ApplyModal.tsx도 영향 없는지 확인함)

Prettier/ESLint/tsc -b/vite build 재확인 통과했습니다.

@kcleverp
kcleverp merged commit 486617a into dev Aug 12, 2026
3 checks passed
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.

1 participant