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: TableRadioFilter 컴포넌트를 추가한다 #942

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

YimJiYoung
Copy link
Contributor

@YimJiYoung YimJiYoung commented Feb 28, 2024

  • TableRadioFilter 컴포넌트 추가
  • TableFilterGroup에서 TableDate/String/MultiFilter의 useEffect에서 change 핸들러를 호출하여 초기에 onFilterChange 함수가 여러번 호출되고 있던 것을 첫 렌더링으로 인한 effect에서는 호출되지 않도록 합니다.
  • TableFilterGroup에서 필터 추가하거나 삭제 시 해당 필터 값이 제대로 반영된 상태로 onFilterChange 함수가 호출되도록 합니다.
  • TableDateFilter에서 날짜 선택 시 중복으로 onFilterChange 함수가 호출되지 않도록 합니다.
image

@YimJiYoung YimJiYoung requested a review from a team February 28, 2024 06:43
Copy link

vercel bot commented Feb 28, 2024

@YimJiYoung is attempting to deploy a commit to the Class101 Team on Vercel.

To accomplish this, @YimJiYoung needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

kang-heewon
kang-heewon previously approved these changes Mar 15, 2024
Copy link

vercel bot commented Jul 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
opensource ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 8, 2024 11:53am

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.

3 participants