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.

Sign up for GitHub

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

Jump to bottom

version 2.6.0 #137

Merged
merged 4 commits into from
Feb 18, 2024
Merged

version 2.6.0 #137

merged 4 commits into from
Feb 18, 2024

Conversation

zbqmgldjfh
Copy link
Member

TODO

헥사고날 아키텍처 적용

@zbqmgldjfh
[�Feat] Hexagonal architecture #129 (#131)
f5f2faa 
* setting(archUnit): 아키텍처 테스팅 의존성 추가

* test(HexagonalArchitecture): User의 HexagonalArchitecture 검증

* refactor: User 도메인 헥사고날 아키텍처로 전환

* feat: 헥사고날 애너테이션 적용

* refactor: API가 포트에 의존하도록 변경

* refactor: UseCase의 구현체들이 Port에 의존하도록 리팩토링

* test: Notice의 HexagonalArchitecture 검증

* refactor: Notice 도메인 헥사고날 아키텍처로 전환

* test: Admin의 HexagonalArchitecture 검증

* refactor(NoticeUpdateSupport): NoticeUpdateSupport 일부 로직 수정

* refactor: Admin 도메인 헥사고날 아키텍처로 전환

* refactor: 필요없는 설정파일 삭제

* test: Staff의 HexagonalArchitecture 검증

* refactor: Staff 도메인 헥사고날 아키텍처로 전환

* feat: User 도메인의 Firebase 로직을 Event로 처리하도록 구현

* feat: User-Token을 preHandler로 검증하도록 변경

* feat(MessageUserEventListener): Token Validation 리스너 제거

* feat(FirebaseService): 비즈니스 로직이 port에 의존하도록 리팩토링

* feat(FirebaseWithAdminUseCase): 어드민의 공지 전송 작업을 이벤트로 구현

* refactor: 기존의 단일 FirebaseService 를 구독과 공지 전송 2개의 서비스로 분리한다

* refactor(DependencyRuleTests): 기존 아키텍처 검증에서 event 페키지 추가 검증하도록 구현

* fix: QueryDsl이 사용하는 dto의 경로 재설정

* chore: 테스트 서식 지정자 제거와 주석 제거

* docs(README): 문서 업데이트

* refactor(ServerProperties): 공통으로 사용중인 서버의 환경변수를 Common 패키지 하부로 이동

* refactor(FirebaseExceptionHandler): exception handler의 페키지를 adapter.out으로 이동
@zbqmgldjfh
[Fix] FirebaseSubscribeService의 토큰 검증 로직 변경 (#133)
1967300 
verifyIdToken의 경우 클라이언트 측에서 별도의 ID 토큰을 생성하여 전달해주어야 하기 때문에 일단 기존의 방식으로 변경한다
@zbqmgldjfh
fix(NoticeQueryApiV2#getNotices): 학과공지 조회에서 important필드 결여로 인한 문제 해결 …
e08b5ab 
…완료 (#135)
@zbqmgldjfh
[Feat] Api docs v2 #125 (#136)
a126209 
* setting: Swagger 의존성 추가

* feat: API 문서화 완료

* setting: Swagger, TestContainer 버전 변경

* setting: SpringBoot version migration from 2.5.5 to 2.7.18

* feat: 문서화에서 제외할 Controller들에게 Hidden 추가

* setting: 사용하지 않는 gradle의 과정 제거
@zbqmgldjfh zbqmgldjfh added the 🚀 Deploy Production deploy production server label Feb 18, 2024
@zbqmgldjfh zbqmgldjfh self-assigned this Feb 18, 2024
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
74.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@github-actions GitHub Actions
Copy link

Unit Test Results

  34 files  +  3    34 suites  +3   28s ⏱️ +7s
142 tests +10  142 ✔️ +10  0 💤 ±0  0 ❌ ±0 
145 runs  +10  145 ✔️ +10  0 💤 ±0  0 ❌ ±0 

Results for commit a126209. ± Comparison against base commit c2b35e9.

This pull request removes 2 and adds 12 tests. Note that renamed tests count towards both. 
com.kustacks.kuring.notice.repository.NoticeRepositoryTest ‑ 사용자가 북마크해둔 공지의 ID로 해당 공지들을 찾아올 수 있다
com.kustacks.kuring.user.repository.UserRepositoryTest ‑ 사용자가 작성한 피드백을 페이징 처리하여 가져올 수 있다

com.kustacks.kuring.acceptance.AuthAcceptanceTest ‑ [v2] 사용자가 잘못된 FCM token으로 요청을 보내면 예외가 발생한다
com.kustacks.kuring.archunit.DependencyRuleTests ‑ Admin 아키텍처 검증
com.kustacks.kuring.archunit.DependencyRuleTests ‑ Notice 아키텍처 검증
com.kustacks.kuring.archunit.DependencyRuleTests ‑ Staff 아키텍처 검증
com.kustacks.kuring.archunit.DependencyRuleTests ‑ User 아키텍처 검증
com.kustacks.kuring.archunit.DependencyRuleTests ‑ 테스트 페키지 의존성 검증
com.kustacks.kuring.message.adapter.in.event.MessageAdminEventListenerTest ‑ 어드민이 커스텀으로 생성한 알림을 전송할 수 있다
com.kustacks.kuring.message.adapter.in.event.MessageAdminEventListenerTest ‑ 어드민이 커스텀으로 생성한 테스트 알림을 전송할 수 있다
com.kustacks.kuring.message.adapter.in.event.MessageUserEventListenerTest ‑ 사용자는 알림을 받고싶은 공지의 카테고리를 구독 취소 수 있다
com.kustacks.kuring.message.adapter.in.event.MessageUserEventListenerTest ‑ 사용자는 알림을 받고싶은 공지의 카테고리를 구독할 수 있다
…

@zbqmgldjfh zbqmgldjfh merged commit 738fdbe into main Feb 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@zbqmgldjfh zbqmgldjfh

Labels
🚀 Deploy Production deploy production server
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zbqmgldjfh