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

Feat: 사전경매 페이지 footer 구현 #102

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Conversation

luminox93
Copy link
Contributor

💡 작업 내용

  • �판매자/구매자일 경우 각각의 footer 가 다르게 나오도록 구현

💡 자세한 설명

  • �판매자일 경우 좋아요를 누른 사람의 수 / 경매로 전환하기 버튼이 노출되도록 구현
  • 구매자일 경우 알람 받기 버튼 노출되도록 구현

📗 참고 자료 (선택)

📢 리뷰 요구 사항 (선택)

🚩 후속 작업 (선택)

  • [] 판매자일 경우 경매로 전환하기 버튼을 누르면 실제 경매로 해당 아이템이 전환되도록 구현
  • [] 판매자일 경우 수정하기 버튼을 누를 경우 아이템 수정이 가능하도록 구현
  • [] 구매자일 경우 좋아요 버튼을 누를 수 있도록 구현

✅ 셀프 체크리스트

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

closes #이슈번호

1. 판매자일 경우 좋아요 카운트 수와 경매로 전환하기 버튼이 보이도록 수정
2. 구매자일 경우 경매 알람 받기 버튼 보이도록 수정
1. remainigBidCount props를 동적으로 받을 수 있도록 수정
2. remainingBidCount가 없을 경우를 대비하여 조건문 수정
@luminox93 luminox93 added ✨feature 구현, 개선 사항 관련 부분 👩🏻‍💻frontend 프론트엔드 작업 labels Oct 4, 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.

확인했습니다! 고생많으셨어요👍

const [isDeleteConfirmOpen, setIsDeleteConfirmOpen] = useState(false);
const [isDeleteSuccessOpen, setIsDeleteSuccessOpen] = useState(false);

const navigate = useNavigate();
console.log(preAuctionDetails);
Copy link
Contributor

Choose a reason for hiding this comment

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

console.log()가 남아있습니다!

@CLOUDoort CLOUDoort merged commit cdb7d83 into dev Oct 4, 2024
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