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 : mainheader uncatch type err 문제 #481

Merged
merged 2 commits into from
Mar 12, 2024

Conversation

bluedog129
Copy link
Contributor


✅ 풀_리퀘스트 체크리스트

  • PR 제목: [feat/fix/refactor...] 작업 내용 한 줄 요약 (브랜치 이름) #이슈번호
  • commit message 가 적절한지 확인해주세요.
  • 적절한 branch 로 요청했는지 확인해주세요.
  • Assignees, Label 을 붙여주세요.
  • 주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer 로 등록해주세요.
  • PR이 승인된 경우 해당 브랜치는 삭제해 주세요!

🔄 변경 사항

무엇을 합니다!

  • NULL이나 undefined가 올 때 빈 배열로 상수를 지정해주는 코드 추가

📎 변경한 이유

  • uncatch type 즉 NULL이나 undefined가 slice함수로 적용되었을 때 err가 발생하는 문제가 생겼음

closes: #479

👨🏻‍💻 테스트 체크리스트

  • 친구가 있었던 상태, 없는 상태 두 케이스를 다 확인해서 slice에서 에러가 안 생기는지 확인

@bluedog129 bluedog129 added the frontend frontend 관련 PR label Mar 12, 2024
@bluedog129 bluedog129 self-assigned this Mar 12, 2024
@bluedog129 bluedog129 linked an issue Mar 12, 2024 that may be closed by this pull request
@bluedog129 bluedog129 marked this pull request as ready for review March 12, 2024 06:43
@donghyun1998
Copy link
Contributor

null에서 slice 메서드 호출하는 이슈 잘 고쳐주신거 확인했습니다. 고생하셨습니다

@bluedog129 bluedog129 merged commit 37b9a4b into main Mar 12, 2024
1 check passed
@bluedog129 bluedog129 deleted the 479-fix-mainheader-uncatch-type-err-문제 branch March 12, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend frontend 관련 PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix : mainheader uncatch type err 문제
2 participants