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

refactor: 경매 관련 API 수정 #41

Merged

Conversation

viaunixue
Copy link
Member

@viaunixue viaunixue commented Sep 10, 2024

#️⃣연관된 이슈

#CHZZ-93 #done

📝작업 내용

  1. 카테고리없이 사전 경매 목록 조회 수정
  2. 내 프로필 조회 API 리팩토링
  3. 내가 참여한 경매 갯수 api 수정
  4. 베스트, 종료임박 5개로 수정
  5. 판매자 입찰 화면에 제공되는 경매 간단 상세 조회 API 구현
  6. 내가 등록한 경매 갯수 API 추가

✅테스트 결과

image

🙏리뷰 요구사항(선택)

- 카테고리 값 필수 값에서 선택 값으로 변경
- 카테고리 없는 경우 최대 5개만 반환
- repository 내 로직 service 단으로 이동
- AuctionParticipationResponse 메소드 추가
- 실제 카운트 값 테스트를 위해 테스트 경매 상품 추가
@viaunixue viaunixue self-assigned this Sep 10, 2024
Copy link
Collaborator

@junest66 junest66 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Collaborator

@junest66 junest66 left a comment

Choose a reason for hiding this comment

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

관련 API 인증 설정을 SecurityConfig에 추가하셔야할것같습니다!

@YeaChan05 YeaChan05 merged commit bf18809 into JECT-Study:develop Sep 22, 2024
1 check passed
@junest66 junest66 added 👨🏻‍💻backend 백엔드 작업 🪄refactor 리팩토링 labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👨🏻‍💻backend 백엔드 작업 🪄refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants