Skip to content

Comments

[fix ]auction list 스켈레톤 맞춤#485

Merged
dolmeengii merged 2 commits intoKernel360:developfrom
geumyoung00:fix/auction-skeleton-#477
Aug 5, 2025
Merged

[fix ]auction list 스켈레톤 맞춤#485
dolmeengii merged 2 commits intoKernel360:developfrom
geumyoung00:fix/auction-skeleton-#477

Conversation

@geumyoung00
Copy link
Collaborator

@geumyoung00 geumyoung00 commented Aug 5, 2025

🚀 풀 리퀘스트 제안

closes #477

📋 작업 내용

  • auction-card-skeleton 사이즈 맞춤
  • categries-filter skeleton 사이즈 맞춤

🔧 변경 사항

  • 📃 README.md
  • 📦 package.json
  • 🔥 파일 삭제
  • 🧹 그 외 ex) .gitignore 등

주요 변경 사항을 요약해 주세요.

📸 스크린샷 (선택 사항)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

📄 기타

추가적으로 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

Sourcery 요약

경매 컴포넌트 전반의 스켈레톤 UI 스타일을 개선하고, 불필요한 클래스와 코드를 정리하며, 컴포넌트 export를 단순화합니다.

개선 사항:

  • 경매 카드 스켈레톤의 크기와 간격을 업데이트된 디자인 요구사항에 맞춰 조정
  • 카테고리 필터 스켈레톤 여백을 업데이트하고, pill 스타일링을 위해 rounded-md 적용
  • 경매 카드 컴포넌트에서 사용되지 않는 thumbnail 클래스와 주석 처리된 BadgeBidStatus 코드 제거
  • 경매 목록 로딩 스켈레톤의 불필요한 간격 제거

기타 작업:

  • 프로필 페이지에서 force-dynamic export 제거
Original summary in English

Summary by Sourcery

Refine skeleton UI styling across auction components, clean up redundant classes and code, and simplify component exports

Enhancements:

  • Align auction card skeleton dimensions and spacing to updated design requirements
  • Update categories filter skeleton margins and apply rounded-md for pill styling
  • Remove unused thumbnail class and commented BadgeBidStatus code from auction card component
  • Eliminate extra gap in auction list loading skeleton

Chores:

  • Remove force-dynamic export from profile page

@sourcery-ai
Copy link
Contributor

sourcery-ai bot commented Aug 5, 2025

검토자 가이드

이 PR은 경매 목록 스켈레톤과 카테고리 필터 스켈레톤의 크기, 여백, 테두리 반경을 조정하여 개선하고, 관련 컴포넌트에서 더 이상 사용되지 않는 클래스와 내보내기를 제거합니다.

파일 수준 변경 사항

변경 사항 세부 정보 파일
경매 목록 전반에 걸쳐 스켈레톤 컴포넌트 스타일 개선
  • 경매 카드 스켈레톤 래퍼에 하단 여백 추가
  • 이미지 컨테이너 크기 및 플레이스홀더 높이 조정
  • 경매 목록 스켈레톤 항목 간 간격 제거
apps/web/src/components/auction/auction-card-skeleton.tsx
apps/web/src/components/common/auction-card/list.tsx
카테고리 필터 스켈레톤 간격 및 모서리 둥글게 조정
  • 필터 스켈레톤 래퍼의 상단 및 하단 여백 조정
  • 플레이스홀더 테두리 반경을 rounded-md로 변경
apps/web/src/components/auction/categories-filter.tsx
사용되지 않는 클래스 및 내보내기 정리
  • 경매 카드에서 사용되지 않는 썸네일 크기 클래스 제거
  • 주석 처리된 BadgeBidStatus 스텁 삭제
  • 프로필 페이지에서 force-dynamic 내보내기 제거
apps/web/src/components/common/auction-card/card.tsx
apps/web/app/profile/page.tsx

연결된 이슈에 대한 평가

이슈 목표 해결 여부 설명
#477 경매 카드 스켈레톤 크기가 실제 경매 카드 크기와 일치하는지 확인합니다.

관련 가능성이 있는 이슈


팁 및 명령어

Sourcery와 상호작용하기

  • 새로운 검토 트리거: 풀 리퀘스트에 @sourcery-ai review라고 댓글을 남기세요.
  • 논의 계속: Sourcery의 검토 댓글에 직접 답글을 달아주세요.
  • 검토 댓글에서 GitHub 이슈 생성: Sourcery에게 검토 댓글에 답글을 달아 이슈를 생성하도록 요청하세요. 또한 검토 댓글에 @sourcery-ai issue라고 답글을 달아 이슈를 생성할 수도 있습니다.
  • 풀 리퀘스트 제목 생성: 언제든지 제목을 생성하려면 풀 리퀘스트 제목 아무 곳에나 @sourcery-ai를 작성하세요. 또한 풀 리퀘스트에 @sourcery-ai title이라고 댓글을 달아 언제든지 제목을 (재)생성할 수 있습니다.
  • 풀 리퀘스트 요약 생성: 원하는 위치에 언제든지 PR 요약을 생성하려면 풀 리퀘스트 본문 아무 곳에나 @sourcery-ai summary를 작성하세요. 또한 풀 리퀘스트에 @sourcery-ai summary라고 댓글을 달아 언제든지 요약을 (재)생성할 수 있습니다.
  • 검토자 가이드 생성: 풀 리퀘스트에 @sourcery-ai guide라고 댓글을 달아 언제든지 검토자 가이드를 (재)생성할 수 있습니다.
  • 모든 Sourcery 댓글 해결: 풀 리퀘스트에 @sourcery-ai resolve라고 댓글을 달아 모든 Sourcery 댓글을 해결하세요. 모든 댓글을 이미 처리하여 더 이상 보고 싶지 않을 때 유용합니다.
  • 모든 Sourcery 검토 해제: 풀 리퀘스트에 @sourcery-ai dismiss라고 댓글을 달아 기존의 모든 Sourcery 검토를 해제하세요. 새로운 검토를 처음부터 시작하고 싶을 때 특히 유용합니다. 새로운 검토를 트리거하려면 @sourcery-ai review라고 댓글을 다는 것을 잊지 마세요!

경험 사용자 지정하기

대시보드에 접속하여 다음을 수행하세요:

  • Sourcery에서 생성한 풀 리퀘스트 요약, 검토자 가이드 등 검토 기능을 활성화하거나 비활성화합니다.
  • 검토 언어를 변경합니다.
  • 사용자 지정 검토 지침을 추가, 제거 또는 편집합니다.
  • 다른 검토 설정을 조정합니다.

도움 받기

Original review guide in English

Reviewer's Guide

This PR refines the auction listing skeleton and category filter skeleton by adjusting sizes, margins, and border radii, and also removes stale classes and exports in related components.

File-Level Changes

Change Details Files
Refined skeleton component styling across auction listing
  • Added bottom margin to auction card skeleton wrapper
  • Adjusted image container size and placeholder heights
  • Removed gap between items in the auction list skeleton
apps/web/src/components/auction/auction-card-skeleton.tsx
apps/web/src/components/common/auction-card/list.tsx
Tweaked category filter skeleton spacing and rounding
  • Adjusted top and bottom margins on filter skeleton wrapper
  • Changed placeholder border radius to rounded-md
apps/web/src/components/auction/categories-filter.tsx
Cleaned up unused classes and exports
  • Removed unused Thumbnail size class in auction card
  • Deleted commented BadgeBidStatus stub
  • Removed force-dynamic export from profile page
apps/web/src/components/common/auction-card/card.tsx
apps/web/app/profile/page.tsx

Assessment against linked issues

Issue Objective Addressed Explanation
#477 Ensure that the auction card skeleton size matches the actual auction card size.

Possibly linked issues


Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

안녕하세요 @geumyoung00 - 변경 사항을 검토했습니다. 몇 가지 피드백이 있습니다:

  • auction-card-skeleton의 새로운 사용자 지정 크기 유틸리티(예: size-21.5)가 Tailwind 구성과 일치하고 다른 스켈레톤과 시각적 일관성을 유지하는지 확인하세요.
  • AuctionItemList의 로딩 상태에서 gap-4 제거를 다시 확인하여 스켈레톤 카드가 여전히 의도한 세로 간격을 가지고 있는지 확인하세요.
  • profile/page.tsx에서 export const dynamic = 'force-dynamic'을 제거하면 렌더링 전략이 변경됩니다. 이 변경 사항이 캐싱 및 SSR 요구 사항과 일치하는지 확인하십시오.
AI 에이전트용 프롬프트
이 코드 검토의 의견을 처리하십시오:
## 전체 의견
- auction-card-skeleton의 새로운 사용자 지정 크기 유틸리티(예: size-21.5)가 Tailwind 구성과 일치하고 다른 스켈레톤과 시각적 일관성을 유지하는지 확인하세요.
- AuctionItemList의 로딩 상태에서 gap-4 제거를 다시 확인하여 스켈레톤 카드가 여전히 의도한 세로 간격을 가지고 있는지 확인하세요.
- profile/page.tsx에서 `export const dynamic = 'force-dynamic'`을 제거하면 렌더링 전략이 변경됩니다. 이 변경 사항이 캐싱 및 SSR 요구 사항과 일치하는지 확인하십시오.

Sourcery는 오픈 소스에 무료입니다 - 저희 리뷰가 마음에 드신다면 공유를 고려해주세요 ✨
제가 더 유용해지도록 도와주세요! 각 댓글에 👍 또는 👎를 클릭해 주시면, 해당 피드백을 사용하여 리뷰를 개선하겠습니다.
Original comment in English

Hey @geumyoung00 - I've reviewed your changes - here's some feedback:

  • Verify that the new custom size utilities (e.g. size-21.5) in the auction-card-skeleton align with your Tailwind config and maintain visual consistency with other skeletons.
  • Double-check the removal of gap-4 in AuctionItemList's loading state to ensure skeleton cards still have the intended vertical spacing.
  • Removing the export const dynamic = 'force-dynamic' in profile/page.tsx changes the rendering strategy—please confirm this aligns with your caching and SSR requirements.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Verify that the new custom size utilities (e.g. size-21.5) in the auction-card-skeleton align with your Tailwind config and maintain visual consistency with other skeletons.
- Double-check the removal of gap-4 in AuctionItemList's loading state to ensure skeleton cards still have the intended vertical spacing.
- Removing the `export const dynamic = 'force-dynamic'` in profile/page.tsx changes the rendering strategy—please confirm this aligns with your caching and SSR requirements.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@dolmeengii dolmeengii merged commit 9b605a6 into Kernel360:develop Aug 5, 2025
2 checks passed
@geumyoung00 geumyoung00 deleted the fix/auction-skeleton-#477 branch August 5, 2025 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] auction card 및 card skeleton 수정

2 participants