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-be: 이메일 전송 로직 중 지원자 일괄 조회 로직 구현 #869

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Original issue description

목적

이메일 전송 로직 중 지원자 일괄 조회 개선

작업 세부사항

  • 지원자 일괄 조회 IN 절로 사용하도록 구현

참고 사항

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


VIEW_ALL_APP_01

closes #868

@github-actions github-actions bot added backend 백엔드 refactor 기존 코드 수정 labels Oct 21, 2024
@github-actions github-actions bot added this to the 스프린트 6.0 milestone Oct 21, 2024
@HyungHoKim00 HyungHoKim00 changed the title refactor-be: 이메일 전송 로직 중 지원자 일괄 조회 개선 feat-be: 이메일 전송 로직 중 지원자 일괄 조회 개선 Oct 21, 2024
@HyungHoKim00 HyungHoKim00 marked this pull request as ready for review October 21, 2024 08:50
Copy link
Contributor Author

1729500622.061679

@HyungHoKim00 HyungHoKim00 changed the title feat-be: 이메일 전송 로직 중 지원자 일괄 조회 개선 feat-be: 이메일 전송 로직 중 지원자 일괄 조회 로직 구현 Oct 21, 2024
@@ -147,8 +143,16 @@ public List<Applicant> findAllByProcesses(List<Process> processes) {
.toList();
}

public List<Applicant> findAllByProcess(Process process) {
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

Choose a reason for hiding this comment

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

문제 없어보입니다

Copy link
Contributor Author

📌 Test Coverage Report

Overall Project 78.99%
Files changed 100% 🍏

File Coverage
ApplicantService.java 96.77% 🍏
EmailFacade.java 91.84% 🍏

Copy link
Contributor

@xogns1514 xogns1514 left a comment

Choose a reason for hiding this comment

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

LGTM

@Dobby-Kim Dobby-Kim merged commit 826be35 into be/develop Oct 22, 2024
36 checks passed
@Dobby-Kim Dobby-Kim deleted the be-868-VIEW_ALL_APP_01 branch October 22, 2024 08:05
Dobby-Kim pushed a commit that referenced this pull request Oct 23, 2024
Dobby-Kim pushed a commit that referenced this pull request Oct 23, 2024
Dobby-Kim pushed a commit that referenced this pull request Oct 23, 2024
@Dobby-Kim Dobby-Kim mentioned this pull request Oct 23, 2024
Dobby-Kim added a commit that referenced this pull request Oct 23, 2024
fix-be: applicantCard 조회 시 평가 점수와 갯수가 모든 프로세스에 대해 나오는 버그 수정 (#902)
fix-be: 이메일 인증 시 이미 가입된 이메일은 예외를 던지도록 변경 (#906)
feat-be: RTR 방식 Redis로 변경 (#889)
feat-be: 이메일 전송 템플릿 적용 (#897)
feat-be: 이메일 인증 후, 회원가입 이메일의 인증 여부 확인 (#882)
refactor-be: 식별자에 Tsid 적용 (#856)
feat-be: 이메일 전송 로직 중 지원자 일괄 조회 로직 구현 (#869)
fix-be: github action의 Redis 환경 변수 추가 (#871)
feat-be: 이메일 인증 기능 구현 (#848)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 refactor 기존 코드 수정
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants