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

Feature/#310 사이드바 현재 보고 있지 않은 팀 이름이 주황색으로 표시되는 버그 수정 #311

Conversation

yeonddori
Copy link
Collaborator

관련 이슈

작업 요약

  • 사이드바 현재 보고 있지 않은 팀 이름이 주황색으로 표시되는 버그 수정

작업 상세 설명

  • 이 문제는 teamId가 1과 12처럼 숫자가 비슷한 경우에만 발생했는데, 그 원인은 currentPath.includes(teamPath) 때문이었습니다. includes 메서드는 부분 문자열만 일치해도 true를 반환하기 때문에, /team/1 경로가 /team/12와도 일치한다고 잘못 인식된 것입니다.
  • 따라서 includes 대신 startsWith를 사용하여 경로가 정확히 일치하는지 확인하도록 변경했습니다.

리뷰 요구 사항

  • 리뷰 예상 시간: 1분

@yeonddori yeonddori added 🐞 버그 예기치 못한 동작 발생 🎨 디자인 CSS 등 사용자 UI 디자인 변경 labels Sep 22, 2024
@yeonddori yeonddori self-assigned this Sep 22, 2024
@yeonddori yeonddori removed the 🎨 디자인 CSS 등 사용자 UI 디자인 변경 label Sep 22, 2024
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

이 문제는 teamId가 1과 12처럼 숫자가 비슷한 경우에만 발생했는데, 그 원인은 currentPath.includes(teamPath) 때문이었습니다. includes 메서드는 부분 문자열만 일치해도 true를 반환하기 때문에, /team/1 경로가 /team/12와도 일치한다고 잘못 인식된 것입니다.
따라서 includes 대신 startsWith를 사용하여 경로가 정확히 일치하는지 확인하도록 변경했습니다.

이욜 자세한 설명까지!
수고하셨습니다!!

Copy link
Collaborator

@mun-kyeong mun-kyeong left a comment

Choose a reason for hiding this comment

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

코드 추가설명 감사합니다~ 수고하셨어요 :)

Copy link
Collaborator

@jasper200207 jasper200207 left a comment

Choose a reason for hiding this comment

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

👍

@yeonddori yeonddori merged commit 23ce519 into develop Sep 27, 2024
1 check passed
@yeonddori yeonddori deleted the Feature/#310-사이드바_현재_보고_있지_않은_팀_이름이_주황색으로_표시되는_버그_수정 branch September 27, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 버그 예기치 못한 동작 발생
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 사이드바 현재 보고 있지 않은 팀 이름이 주황색으로 표시되는 버그 수정
4 participants