Skip to content

Conversation

@jwooo002
Copy link
Contributor

@jwooo002 jwooo002 commented Aug 25, 2025

📝 Pull Request Template

📌 제목

notice 도메인 개발


📢 요약

글 작성, 글 수정, 글 상세 조회, 글 목록 조회, 글 삭제 구현

🔗 연관 이슈: Resolves #이슈번호
#14
#15

🚀 PR 유형

  • ✨ 새로운 기능 추가
  • 🐛 버그 수정
  • 🎨 CSS/UI 디자인 변경
  • 🔧 코드에 영향 없는 변경(오타 수정, 탭 사이즈 변경, 변수명 변경 등)
  • 🔨 코드 리팩토링
  • 📝 주석 추가 및 수정
  • 📄 문서 수정
  • 🧪 테스트 추가 또는 리팩토링
  • 🏗️ 빌드 및 패키지 매니저 수정
  • 📂 파일 또는 폴더명 수정
  • 🗑️ 파일 또는 폴더 삭제

✅ PR 체크리스트

  • 🔹 커밋 메시지 컨벤션을 준수했습니다. (Commit message convention 참고)
  • 🔹 변경 사항에 대한 테스트를 수행했습니다. (버그 수정/기능 테스트)
  • 🔹 관련 문서를 업데이트했습니다. (필요한 경우)

📜 기타

삭제할 때 소프트 삭제를 해야하나 하드 삭제를 해야하나 잘 모르겠네요 테이블 별로 다를 거 같긴 한데
지금까지 제가 개발한 도메인에 있는 테이블들은 모두 하드 삭제 했어요

@jwooo002 jwooo002 requested a review from KoungQ August 25, 2025 14:22
@jwooo002 jwooo002 self-assigned this Aug 25, 2025
@jwooo002 jwooo002 linked an issue Aug 25, 2025 that may be closed by this pull request
8 tasks
return BaseResponse.onSuccess(loadNoticesUseCase.loadNotices(userNo, roomNo));
}

@GetMapping("/api/notice")
Copy link
Member

Choose a reason for hiding this comment

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

/api/notices/{noticeNo}


private final DeleteNoticeUseCase deleteNoticeUseCase;

@DeleteMapping("/api/notice")
Copy link
Member

Choose a reason for hiding this comment

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

/api/notices/{noticeNo}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] notice 기능 개발

3 participants