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

ES 쿼리 로직 수정 #1523

Merged
merged 2 commits into from
Sep 4, 2023
Merged

ES 쿼리 로직 수정 #1523

merged 2 commits into from
Sep 4, 2023

Conversation

mcodnjs
Copy link
Contributor

@mcodnjs mcodnjs commented Sep 4, 2023

#️⃣연관된 이슈

연관된 이슈 번호를 모두 작성해주세요
#1522

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@sonarcloud
Copy link

sonarcloud bot commented Sep 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

wonyongChoi05
wonyongChoi05 approved these changes Sep 4, 2023
View reviewed changes
Copy link
Member

@wonyongChoi05 wonyongChoi05 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

화이팅!!

This2sho
This2sho approved these changes Sep 4, 2023
View reviewed changes
Copy link
Contributor

@This2sho This2sho left a comment

Choose a reason for hiding this comment

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

🥺

@mcodnjs mcodnjs merged commit 57a87a4 into main Sep 4, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@This2sho This2sho This2sho approved these changes

@wonyongChoi05 wonyongChoi05 wonyongChoi05 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mcodnjs @This2sho @wonyongChoi05