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

fix(.plan): 리마인드 메세지 정보 조회 시 프록시 오류 발생 문제 수정 #187

Closed
wants to merge 1 commit into from
Closed

fix(.plan): 리마인드 메세지 정보 조회 시 프록시 오류 발생 문제 수정 #187

wants to merge 1 commit into from

Conversation

2jie0516
Copy link
Contributor

🔍 어떤 PR인가요?

  • 리마인드 전송 시에 프록시 오류가 발생하던 문제를 수정하였습니다.
  • 트랜잭션을 걸어주어서 lazy loading 방식인 리마인드 메세지들도 조회할 수 있도록 하였습니다.

@2jie0516 2jie0516 self-assigned this Feb 28, 2024
@github-actions GitHub Actions
Copy link

Test Coverage Report

Overall Project 80.75% -0.42% 🍏
Files changed 82.84% 🍏

File Coverage
FindRemindablePlanAdapter.java 81.3% -18.7% 🍏
FindRemindableTargetAdapter.java 68.42% 🍏

@2jie0516 2jie0516 closed this Feb 28, 2024
@Hejow Hejow deleted the fix-sehan branch February 29, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hejow Hejow Awaiting requested review from Hejow Hejow is a code owner

@kys0411 kys0411 Awaiting requested review from kys0411 kys0411 is a code owner

@JuwoongKim JuwoongKim Awaiting requested review from JuwoongKim JuwoongKim is a code owner

Assignees

@2jie0516 2jie0516

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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