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

[Fix] 선착순 미참여시 최종 결과 페이지 접근 오류 #194

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

sooyeoniya
Copy link
Member

🖥️ Preview

(없음)
close #192

✏️ 한 일

  • 선착순 미참여시 최종 결과 페이지 접근 오류 처리

❗️ 발생한 이슈 (해결 방안)

function formatNumber(value?: number): string {
    if (value === undefined || value === null) return "";
    return value.toLocaleString("en-US");
}

const formattedRank = formatNumber(rank);
const formattedTotalParticipants = formatNumber(totalParticipants);

최종 결과를 보여주는 화면에서 rank, totalParticipants가 null일 경우가 존재하지 않는다고 생각했다.
실제로 타입을 다음과 같이 정의해놨고 타입 추론도 number | undefined였기 때문이다.

export interface GetRushResultResponse {
    optionId?: CardOption;
    isWinner?: boolean;
    leftOption: number;
    rightOption: number;
    rank?: number;
    totalParticipants?: number;
}

근데 보아하니 두 값들을 가져오는 resultData 자체가 null일 가능성이 존재했다.
따라서 formatNumber 함수의 value 부분에 null도 추가로 처리해주었다.

❓ 논의가 필요한 사항

@sooyeoniya sooyeoniya added the fix 버그가 발생 label Aug 22, 2024
@sooyeoniya sooyeoniya requested a review from jhj2713 August 22, 2024 07:13
@sooyeoniya sooyeoniya self-assigned this Aug 22, 2024
Copy link

빌드를 성공했습니다! 🎉

@jhj2713
Copy link
Member

jhj2713 commented Aug 22, 2024

굿굿👍

@jhj2713 jhj2713 merged commit 7c05761 into dev Aug 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그가 발생
Projects
None yet
Development

Successfully merging this pull request may close these issues.

선착순 미참여시 최종 결과 페이지 접근 오류
2 participants