Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

게시글(공지) 관련 API 구현 #312

Closed
9 tasks done
kdkdhoho opened this issue Oct 14, 2024 · 0 comments · Fixed by #323
Closed
9 tasks done

게시글(공지) 관련 API 구현 #312

kdkdhoho opened this issue Oct 14, 2024 · 0 comments · Fixed by #323
Assignees
Labels

Comments

@kdkdhoho
Copy link
Collaborator

kdkdhoho commented Oct 14, 2024

Todo

  • 공지 카테고리 조회 API
  • 생성 API
  • 어드민용 전체 조회 API
  • 사용자용 전체 조회 API
  • 상세 조회 API
  • 내용 수정 API
  • 공개 여부 수정 API
  • 삭제 API
  • 공지 알람 API 구현 #316
@kdkdhoho kdkdhoho self-assigned this Oct 14, 2024
@kdkdhoho kdkdhoho changed the title 게시글(공지) API 게시글(공지) CRUD API Oct 14, 2024
@kdkdhoho kdkdhoho changed the title 게시글(공지) CRUD API 게시글(공지) 관련 API 구현 Oct 29, 2024
@kdkdhoho kdkdhoho pinned this issue Oct 29, 2024
kdkdhoho added a commit that referenced this issue Oct 30, 2024
정확한 원인은 파악하지 못했지만, 추측컨데 AcceptanceTest와 IntegrationTest의 테스트 DB가 공유되면서 외래키 제약 조건으로 발생한 것으로 예상 (#312)
@kdkdhoho kdkdhoho linked a pull request Oct 30, 2024 that will close this issue
@kdkdhoho kdkdhoho unpinned this issue Oct 30, 2024
kdkdhoho added a commit that referenced this issue Oct 30, 2024
kdkdhoho added a commit that referenced this issue Oct 30, 2024
* test: 전체 테스트가 실패하는 문제 해결

정확한 원인은 파악하지 못했지만, 추측컨데 AcceptanceTest와 IntegrationTest의 테스트 DB가 공유되면서 외래키 제약 조건으로 발생한 것으로 예상 (#312)

* test: 요청 주제(토픽) 초기 생성 시 isExposed를 항상 true로 변경함에 따른 테스트 코드 수정 (#312)

* feat: 공지 카테고리(타입) 조회 API 구현 (#312)

* feat: Notice 도메인 객체 생성 (#312)

* feat: 공지 생성 API 구현 (#312)

* feat: 사용자용, 관리자용 공지 전체 조회 및 공지 상세 조회 API 구현 (#312)

* feat: 공지 수정 API 구현 (#312)

* feat: 공지 삭제 API 구현 (#312)

* fix: 관리자 존재 체크 로직 제거 (#312)

* refactor: 빠진 @nullable 추가 (#312)

* test: NoticeCreateRequest 생성 부분 리팩터링 (#312)
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 a pull request may close this issue.

1 participant