Skip to content

fix(fe): prevent contest delete modal from opening without selection#3635

Draft
jinukkkim wants to merge 1 commit into
mainfrom
t2759-contest-bug-fix
Draft

fix(fe): prevent contest delete modal from opening without selection#3635
jinukkkim wants to merge 1 commit into
mainfrom
t2759-contest-bug-fix

Conversation

@jinukkkim

@jinukkkim jinukkkim commented Jul 3, 2026

Copy link
Copy Markdown

Description

어드민 페이지의 데이터 테이블에서, 선택된 항목이 없을 때 휴지통 버튼을 누르면 에러 토스트( Please select at least one... )와 함께 삭제 경고 모달 창이 동시에 강제로 열려버리는 버그를 수정했습니다.

Additional context


Before submitting the PR, please make sure you do the following

Closes TAS-2759

선택된 항목이 없을 때 휴지통 버튼을 누르면 모달이 강제로 열리는 버그를 수정한다.
AlertModal의 트리거 버튼을 분리하여 모달 상태 제어를 독립시킨다.
@jinukkkim jinukkkim requested review from egg-zz and seoeun9 July 3, 2026 05:29
@jinukkkim jinukkkim self-assigned this Jul 3, 2026
@jinukkkim jinukkkim added ⛳️ team-frontend preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 labels Jul 3, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the DataTableDeleteButton component by rendering the trigger button and the AlertModal side-by-side within a React Fragment, rather than passing the button as a trigger prop. The feedback suggests two improvements: adding an aria-label to the icon-only button to improve web accessibility (a11y), and replacing the object spreading pattern for the conditional description prop with a cleaner ternary operator to enhance readability.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread apps/frontend/app/admin/_components/table/DataTableDeleteButton.tsx
Comment thread apps/frontend/app/admin/_components/table/DataTableDeleteButton.tsx
@skkuding-bot

skkuding-bot Bot commented Jul 3, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 3e6f321943826a1aef77ab19e9c9299eba911d27
Health Status: Healthy

Open Preview | View in Argo CD

@egg-zz egg-zz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM~

@seoeun9 seoeun9 requested a review from HajunPark-skku July 6, 2026 11:19
@HajunPark-skku HajunPark-skku enabled auto-merge July 8, 2026 02:30
@HajunPark-skku HajunPark-skku disabled auto-merge July 8, 2026 02:30
@skkuding-bot skkuding-bot Bot removed the preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 label Jul 8, 2026
@HajunPark-skku HajunPark-skku reopened this Jul 8, 2026

@HajunPark-skku HajunPark-skku left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! 수고하셨습니당

@HajunPark-skku HajunPark-skku self-requested a review July 8, 2026 03:17
@seoeun9 seoeun9 added the preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 label Jul 8, 2026
@skkuding-bot

skkuding-bot Bot commented Jul 8, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 3e6f321943826a1aef77ab19e9c9299eba911d27
Health Status: Healthy

Open Preview | View in Argo CD

@seoeun9 seoeun9 marked this pull request as draft July 8, 2026 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 ⛳️ team-frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants