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

경매 추천 시스템 반영 #200

Merged
merged 3 commits into from
Aug 14, 2024
Merged

Conversation

JadeKim042386
Copy link
Member

@JadeKim042386 JadeKim042386 commented Aug 14, 2024

🔍️ 이 PR을 통해 해결하려는 문제

#195 와 마찬가지로 경매도 추천 시스템 동작을 위해 로직을 추가합니다.

✨ 이 PR에서 핵심적으로 변경된 사항

  • 경매는 추가만 할 수 있기 때문에 추가할 때 경매 프로필을 수정 요청

  • 낙찰된 경매를 결제(구매)할 때 구매한 회원의 프로필을 수정 요청

  • 테스트 코드 수정

  • 경매 추천 요청은 FastAPI 서버로 다음 엔드포인트로 요청하면 됩니다.

    • 엔드포인트: /rec-sys/auctions/2/?limit=10
image

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분

  • 없음

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 1일 이내에 진행해 주세요.

Issue Tags

- 경매는 추가만 할 수 있기 때문에 추가할 때 경매 프로필을 수정 요청
- 낙찰된 경매를 결제(구매)할 때 구매한 회원의 프로필을 수정 요청
@JadeKim042386 JadeKim042386 added the enhancement New feature or request label Aug 14, 2024
@JadeKim042386 JadeKim042386 self-assigned this Aug 14, 2024
Copy link

codecov bot commented Aug 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Files Coverage Δ Complexity Δ
.../domain/alarm/service/CompletePayAuctionAlarm.java 91.66% <100.00%> (+1.66%) 5.00 <1.00> (ø)
...backend/domain/auction/service/AuctionService.java 80.32% <100.00%> (+0.32%) 15.00 <0.00> (ø)
.../freshtrashbackend/global/infra/RecSysService.java 100.00% <100.00%> (ø) 6.00 <2.00> (+2.00)

@JadeKim042386 JadeKim042386 force-pushed the feature/#199-auction-recsys branch 2 times, most recently from 7f71695 to 88bb64e Compare August 14, 2024 08:04
@JadeKim042386 JadeKim042386 merged commit 3597c9e into develop Aug 14, 2024
3 checks passed
@JadeKim042386 JadeKim042386 deleted the feature/#199-auction-recsys branch August 14, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

경매 추천 시스템 반영
1 participant