Skip to content

feat: 자료 수정·파일 교체와 회장 작성자 지정 - #446

Merged
choijunhuk merged 1 commit into
mainfrom
feat/content-edit
Sep 11, 2026
Merged

choijunhuk merged 1 commit into
mainfrom
feat/content-edit

Conversation

@choijunhuk

Copy link
Copy Markdown
Collaborator

변경 사항

  • 자료실 PUT 수정 API와 웹 수정 폼: 기존 id·추천·조회수 유지
  • 파일 교체는 커밋 후 원본 정리, 롤백 시 신규 파일 정리
  • 공지·커뮤니티 내용 수정은 원작성자 또는 회장, 고정·삭제 권한은 기존 정책 유지
  • 작성자 이름 직접 입력과 회원 검색/소유권 이전 분리
  • V91: 공지 작성자 학번 nullable 컬럼, 기존 작성자 추정 없음

검증

백엔드 467건 실패 0, PostgreSQL 전용 1건 로컬 skip(CI 실행).
웹 npm test·빌드 통과. 전체 스모크 57건과 최종 수정 경로 5건 통과.
독립 코드 리뷰 지적 수정 및 재검토 완료.

…rator edit rights

Add archive metadata/file editing and explicit display-name versus member reassignment controls. Keep original ownership on ordinary edits and restrict notice/community content override to the president.

Constraint: Existing ids, engagement counts and attachment safety must survive edits
Rejected: Delete and reupload archive entries | breaks references and engagement history
Directive: Reassign ownership only through the separate author action; legacy notice authors are not inferred
Tested: Backend 467 cases, zero failures, one PostgreSQL-only skip; web lint/typecheck/build, smoke and targeted editing flows
Not-tested: Production rollout and native-device installation
Scope-risk: moderate
@choijunhuk
choijunhuk merged commit 36fcde3 into main Sep 11, 2026
4 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