Skip to content

fix: support additional admin attachment formats - #261

Merged
YoonKeumJae merged 2 commits into
oneot:developfrom
YoonKeumJae:codex/fix-admin-attachment-upload
Jun 17, 2026
Merged

fix: support additional admin attachment formats#261
YoonKeumJae merged 2 commits into
oneot:developfrom
YoonKeumJae:codex/fix-admin-attachment-upload

Conversation

@YoonKeumJae

Copy link
Copy Markdown
Collaborator

Summary

  • add Admin attachment support for .ppt, .hwp, and .hwpx
  • align server-side attachment MIME allowlist with Admin upload UI
  • show truncated upload failure details in Admin so future upload failures are diagnosable

Validation

  • npm test in Elevate.Server
  • npm run lint in Elevate.Admin
  • npm run build in Elevate.Admin
  • local Admin dev-server smoke with Playwright; app redirected to MSAL login as expected

Ready for review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Admin 첨부파일 업로드에서 추가 확장자(.ppt, .hwp, .hwpx)를 지원하고, 서버의 첨부 MIME allowlist를 UI와 맞추며, 업로드 실패 시 Admin 화면에 에러 상세(일부)를 노출해 원인 파악이 가능하도록 개선합니다.

Changes:

  • (Server) 첨부파일 MIME allowlist에 .ppt/한글(HWP/HWPX) 계열 MIME 타입 추가
  • (Server) createFileMetadata에서 신규 MIME 타입을 허용하는 테스트 케이스 추가
  • (Admin) 업로드 가능 확장자/accept 목록을 상수화하고, 업로드 실패 메시지에 에러 메시지(최대 180자)를 포함

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
Elevate.Server/tests/admin-attachments.test.js createFileMetadata.ppt/.hwp/.hwpx MIME을 허용하는지 테스트 추가
Elevate.Server/src/controllers/adminController.js 서버 첨부파일 MIME allowlist에 PowerPoint(HWP/HWPX 포함) 타입 추가
Elevate.Admin/src/components/editor/AttachUploader.jsx Admin 업로드 UI에서 신규 확장자 지원, accept/안내 문구 자동화, 실패 에러 메시지 일부 표시

@YoonKeumJae
YoonKeumJae merged commit 03a4f16 into oneot:develop Jun 17, 2026
@YoonKeumJae
YoonKeumJae deleted the codex/fix-admin-attachment-upload branch June 17, 2026 08:43
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