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

Hotfix | #81 | @YongsHub | v1.0.1 출시 위해 Query Transform 리팩토링 #88

Merged
merged 10 commits into from
Feb 29, 2024

Conversation

YongsHub
Copy link
Contributor

Issue Number

#81

Description

Core Code

etc

@YongsHub YongsHub added v1.0 version 1.0 fix fix wrong logic labels Feb 28, 2024
@YongsHub YongsHub requested a review from lcomment February 28, 2024 18:55
@YongsHub YongsHub self-assigned this Feb 28, 2024
Copy link

github-actions bot commented Feb 28, 2024

Test Results

15 files  ±0  15 suites  ±0   4s ⏱️ ±0s
57 tests ±0  57 ✅ ±0  0 💤 ±0  0 ❌ ±0 
99 runs  ±0  99 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 251ac82. ± Comparison against base commit 1c268a1.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@lcomment lcomment left a comment

Choose a reason for hiding this comment

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

Map에서 dto로 가공할 필요없이 쿼리에서 dto로 바로 넘길 수 있을텐대, 이후에 제가 처리해보겠습니다. 고생하셨어요 !


import com.lovebird.domain.dto.query.DiaryResponseParam

data class DiarySimpleResponse(
Copy link
Contributor

Choose a reason for hiding this comment

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

갑자기 네이밍을 바꾸신 특별한 이유가 있으실까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

DiaryResponse와 DiarySimpleResponse의 return value들이 기존에 달랐던걸 인지하지 못하고 통합해버려서 원래대로 다시 분리했습니다!

@lcomment
Copy link
Contributor

CI에서 패키지 import 순서 관련 ktlint 에러 났네요 ~ 확인해주세요 ~

@YongsHub YongsHub merged commit d0cac56 into develop Feb 29, 2024
2 checks passed
@lcomment lcomment deleted the hotfix/#81 branch February 29, 2024 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix wrong logic v1.0 version 1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants