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

feat: 관심 분야로 멤버 목록을 조회할 수 있도록 쿼리 조건 추가 #8

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

Coalery
Copy link
Member

@Coalery Coalery commented Sep 6, 2023

주요 변경 사항

관심 분야로 멤버 목록을 조회할 수 있도록 쿼리 조건으로 interest를 추가하였습니다.

또한, import 문에 대해 린트를 적용하였습니다.
그리고, AppModule에서 이미 삭제된 서비스인 AppService를 참조하는 부분을 수정하였습니다.

변경 이유

레거시에서 그러한 기능이 있습니다.
그 외 두 가지는 코드 정리 용도입니다.

@Coalery Coalery self-assigned this Sep 6, 2023
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #8 (d10d6db) into main (dd004d8) will increase coverage by 0.22%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   63.75%   63.97%   +0.22%     
==========================================
  Files          11       11              
  Lines         160      161       +1     
  Branches        2        2              
==========================================
+ Hits          102      103       +1     
  Misses         58       58              
Files Changed Coverage Δ
src/__test__/fixtures/view.ts 100.00% <ø> (ø)
...ication/user/query/listUser/ListUserQueryResult.ts 100.00% <ø> (ø)
src/app/domain/user/model/User.ts 55.26% <ø> (ø)
src/__test__/fixtures/domain.ts 100.00% <100.00%> (ø)
...p/application/user/query/listUser/ListUserQuery.ts 100.00% <100.00%> (ø)
...cation/user/query/listUser/ListUserQueryHandler.ts 100.00% <100.00%> (ø)

@Coalery Coalery merged commit bb2bd8b into main Sep 6, 2023
@Coalery Coalery deleted the feat/list-member-with-interest branch September 6, 2023 15:44
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