Skip to content

코드 수정 작업 #504

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

Open
wants to merge 24 commits into
base: dev/client
Choose a base branch
from

Conversation

HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Mar 25, 2023

  • BtnDropDown.tsx 파일에서 toast 적용
  • 우선 mobile: 이라고 적혀져있던 부분은 반응형 할 때 다시 하기 위해 다 삭제했습니다.
  • SearchBar.tsx에서는 button 타입을 명시해주지 않으면 라이트하우스에서 점수가 낮아지는 거 같아서 따로 명시해두었습니다.
  • 프로필 수정 시에 메인으로 가는 부분은 제가 생각했을 때 조금 이상한 거 같아서 toast 컴포넌트로 대체했는데 이 부분은 확인 부탁드려용
  • any로 되어있던 부분은 타입 명시해두었습니다.

@HA-SEUNG-JEONG HA-SEUNG-JEONG requested a review from hyejj19 March 25, 2023 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant