feat: 프로젝트 공지 대시보드 crud 구현 - #25
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthrough프로젝트 공지 기능에 요청·응답 DTO, 엔티티 상태 변경, 활성 공지 저장소, 권한 검증 서비스, 커서 기반 목록 조회 및 CRUD REST API를 추가했다. Changes프로젝트 공지 기능
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Client
participant ProjectNoticeController
participant ProjectNoticeService
participant ProjectNoticeRepository
Client->>ProjectNoticeController: 프로젝트 공지 API 요청
ProjectNoticeController->>ProjectNoticeService: 사용자 ID와 공지 요청 전달
ProjectNoticeService->>ProjectNoticeRepository: 활성 공지 조회 또는 저장
ProjectNoticeRepository-->>ProjectNoticeService: 공지 데이터 반환
ProjectNoticeService-->>ProjectNoticeController: 공지 응답 생성
ProjectNoticeController-->>Client: ApiResponse 반환
Possibly related PRs
🚥 Pre-merge checks | ✅ 2 | ❌ 3❌ Failed checks (3 warnings)
✅ Passed checks (2 passed)
✨ Finishing Touches📝 Generate docstrings
Comment |
chazy-d
requested review from
guingguing,
kcleverp,
sangwon02 and
young0206
and removed request for
guingguing,
kcleverp,
sangwon02 and
young0206
July 20, 2026 04:50
Merged
5 tasks
This was referenced Aug 3, 2026
Merged
Merged
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 관련 이슈 (Related Issue)
📝 작업 내용
프로젝트 대시보드의 공지사항 영역에서 사용할 공지사항 CRUD API를 추가했습니다.
공지사항은 파일 첨부 없이 제목과 내용만 등록하는 구조로 구현했고, 프로젝트 접근 권한과 작성자/관리자 기준의 수정·삭제 권한을 검증하도록 구성했습니다.
주요 검토 파일
주요 변경 파일
src/main/java/com/slatto/domain/project/controller/ProjectNoticeController.java- 공지사항 목록 조회, 생성, 수정, 삭제 엔드포인트 추가src/main/java/com/slatto/domain/project/service/ProjectNoticeService.java- 공지사항 CRUD 비즈니스 로직 추가src/main/java/com/slatto/domain/project/entity/ProjectNotice.java- 공지사항 수정 및 삭제 도메인 메서드 추가src/main/java/com/slatto/domain/project/repository/ProjectNoticeRepository.java- 프로젝트별 공지사항 조회 쿼리 추가src/main/java/com/slatto/domain/project/dto/ProjectNotice*.java- 공지사항 요청/응답 DTO 추가1. 프로젝트 공지사항 목록 조회 API 추가
프로젝트 상세 대시보드에서 공지사항 목록을 조회할 수 있도록 API를 추가했습니다.
프로젝트에 접근 권한이 있는 사용자만 조회할 수 있도록 검증합니다.
삭제된 공지사항은 목록에 포함하지 않습니다.
2. 프로젝트 공지사항 등록 API 추가
프로젝트 공지사항을 등록할 수 있도록 API를 추가했습니다.
요청 본문은 제목과 내용으로 구성했습니다.
{ "title": "공지사항 제목", "content": "공지사항 내용" }공지사항 파일 첨부는 현재 범위에서 제외되어 있어 S3, ProjectFile, 업로드/다운로드 로직은 건드리지 않았습니다.
3. 프로젝트 공지사항 수정 API 추가
등록된 공지사항의 제목과 내용을 수정할 수 있도록 API를 추가했습니다.
수정은 공지 작성자 또는 프로젝트
ADMIN만 가능하도록 검증합니다.4. 프로젝트 공지사항 삭제 API 추가
등록된 공지사항을 삭제할 수 있도록 API를 추가했습니다.
삭제는 실제 row를 제거하지 않고
deletedAt을 기록하는 soft delete 방식으로 처리했습니다.삭제 권한은 수정과 동일하게 공지 작성자 또는 프로젝트
ADMIN기준으로 검증합니다.✅ PR 체크리스트