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: 관심사의 id로 쿼리하도록 수정 #34

Merged
merged 1 commit into from
Oct 28, 2023
Merged

fix: 관심사의 id로 쿼리하도록 수정 #34

merged 1 commit into from
Oct 28, 2023

Conversation

Coalery
Copy link
Member

@Coalery Coalery commented Oct 28, 2023

주요 변경 사항

관심사의 id로 쿼리하도록 수정합니다.

변경 이유

관심사 문자열로 비교하는 것이 아니라 UserInterest 도메인 기준으로 쿼리할 것이기 때문에, interest 대신에 interestId를 받도록 수정합니다.

@Coalery Coalery self-assigned this Oct 28, 2023
@github-actions
Copy link

Coverage report

St.❔
 Category Percentage Covered / Total
🔴 Statements 57.03% 280/491
🔴 Branches 48.33% 29/60
🔴 Functions 31.74% 53/167
🔴 Lines 55.56% 255/459

Test suite run success

25 tests passing in 9 suites.

Report generated by 🧪jest coverage report action from 0c946a1

@Coalery Coalery merged commit 122c031 into main Oct 28, 2023
2 checks passed
@Coalery Coalery deleted the fix/list-user-with-interest-id branch October 28, 2023 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Coalery Coalery

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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