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

fix: 타입 정리 #85

Merged
merged 2 commits into from
Oct 2, 2024
Merged

fix: 타입 정리 #85

merged 2 commits into from
Oct 2, 2024

Conversation

CLOUDoort
Copy link
Contributor

💡 작업 내용

  • auction & pre-auction 타입 정리

💡 자세한 설명

interface로 정의한 타입은 앞에 I 를 붙여서 구분했습니다.
@types 폴더의 AuctionItem과 AuctionDetails에 타입 정리 모두 해놨습니다.
필요하면 가져다가 사용하면 되겠습니다.

✅ 셀프 체크리스트

  • PR 제목을 형식에 맞게 작성했나요?
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요? (master/main이 아닙니다.)
  • 이슈는 close 했나요?
  • Reviewers, Labels, Projects를 등록했나요?
  • 작업 도중 문서 수정이 필요한 경우 잘 수정했나요?
  • 테스트는 잘 통과했나요?
  • 불필요한 코드는 제거했나요?

closes #84

@CLOUDoort CLOUDoort added ✨feature 구현, 개선 사항 관련 부분 👩🏻‍💻frontend 프론트엔드 작업 labels Oct 2, 2024
Copy link
Contributor

@aquaman122 aquaman122 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
Contributor

Choose a reason for hiding this comment

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

인터페이스 정리 확인했습니다
오늘 저녁에 백엔드 분들과 pre-acution 부분 api 협의 + 변경이 있을 것 같은데 해당 부분 추후에 전파드리겠습니다

@CLOUDoort CLOUDoort merged commit 25ee3e2 into dev Oct 2, 2024
@CLOUDoort CLOUDoort deleted the fix/clean-type branch October 5, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨feature 구현, 개선 사항 관련 부분 👩🏻‍💻frontend 프론트엔드 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants