Skip to content

LC-2459-feat-챌린지 대시보드 미션, 콘텐츠 버튼 수정 #1892

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.

Sign up for GitHub

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

Open
yeeun426 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

LC-2459-feat-챌린지 대시보드 미션, 콘텐츠 버튼 수정 #1892

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/components/common/challenge/my-challenge/section/MissionGuideRegularSection.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ const MissionGuideRegularSection = ({
<MissionFileLink
key={missionData?.missionInfo?.templateLink}
title="미션 템플릿"
fileName={'미션 템플릿'}
fileName={'미션 템플릿 확인하기'}
disabled={false}
onClick={() => {
window.open(
Expand All @@ -112,7 +112,7 @@ const MissionGuideRegularSection = ({
<MissionFileLink
key={content.id || index}
title="필수 콘텐츠"
fileName={content.title || '필수 콘텐츠'}
fileName={'필수 콘텐츠 확인하기'}
Copy link

@gemini-code-assist gemini-code-assist bot Sep 19, 2025

Choose a reason for hiding this comment

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

high

fileName을 '필수 콘텐츠 확인하기'라는 정적 문자열로 변경하면, essentialContentsList에 여러 항목이 있을 때 각 링크를 구별할 수 없는 문제가 발생합니다. 예를 들어, 두 개의 다른 필수 콘텐츠가 모두 '필수 콘텐츠 확인하기'라는 버튼으로 표시되어 사용자에게 혼란을 줄 수 있습니다. 이전처럼 각 콘텐츠의 고유한 title을 사용하도록 되돌리는 것을 권장합니다. 만약 "확인하기" 문구를 포함해야 한다면, fileName={content.title ? ${content.title} 확인하기 : '필수 콘텐츠 확인하기'}와 같이 동적으로 구성하는 방법을 고려해볼 수 있습니다.

Suggested change
fileName={'필수 콘텐츠 확인하기'}
fileName={content.title || '필수 콘텐츠'}

disabled={false}
onClick={() => {
window.open(content.link || '#', '_blank');
Expand All @@ -128,7 +128,7 @@ const MissionGuideRegularSection = ({
<MissionFileLink
key={content.id || index}
title={index === 0 ? '추가 콘텐츠' : ''}
fileName={content.title ?? '추가 콘텐츠'}
fileName={'추가 콘텐츠 확인하기'}
Copy link

@gemini-code-assist gemini-code-assist bot Sep 19, 2025

Choose a reason for hiding this comment

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

high

'필수 콘텐츠'와 마찬가지로 additionalContentsList의 fileName을 '추가 콘텐츠 확인하기'라는 정적 문자열로 변경하면, 여러 항목이 있을 때 각 링크를 구별하기 어렵습니다. 각 콘텐츠의 고유한 title을 사용하도록 되돌리는 것이 좋습니다.

Suggested change
fileName={'추가 콘텐츠 확인하기'}
fileName={content.title ?? '추가 콘텐츠'}

disabled={false}
onClick={() => {
window.open(content.link || '#', '_blank');
Expand Down
2 changes: 1 addition & 1 deletion src/components/common/challenge/my-challenge/section/MissionGuideZeroSection.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ const MissionGuideZeroSection = ({
(content, index) => (
<MissionFileLink
key={content.id || index}
fileName={content.title || ''}
fileName={'필수 콘텐츠 확인하기'}
Copy link

@gemini-code-assist gemini-code-assist bot Sep 19, 2025

Choose a reason for hiding this comment

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

high

fileName을 '필수 콘텐츠 확인하기'라는 정적 문자열로 변경하면, essentialContentsList에 여러 항목이 있을 때 각 링크를 구별할 수 없는 문제가 발생합니다. 이전 구현(content.title || '')에서 빈 문자열이 표시되는 것이 문제였다면, content.title || '필수 콘텐츠'와 같이 적절한 대체 텍스트를 제공하면서 각 콘텐츠의 고유한 제목을 유지하는 것이 좋습니다.

Suggested change
fileName={'필수 콘텐츠 확인하기'}
fileName={content.title || '필수 콘텐츠'}

title={index === 0 ? '필수 콘텐츠' : ''}
disabled={false}
onClick={() => {
Expand Down