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

refactor: 로그인 부분 코드 리팩토링 진행 #263

Merged
merged 7 commits into from
Jan 1, 2025

Conversation

urimeee
Copy link
Member

@urimeee urimeee commented Nov 21, 2024

#️⃣연관된 이슈

#259

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

다른 모달과의 구분을 위해 Modal에서 CloseModal로 네이밍을 변경했습니다.
Modal을 컴포넌트화 했습니다.
코드 가독성 향상을 위해 else문에 공백 추가 및 사용하지 않는 함수의 선언 및 사용하지 않는 변수의 삭제를 했습니다.
@urimeee urimeee added the refactor 리팩토링 label Nov 21, 2024
@urimeee urimeee self-assigned this Nov 21, 2024
@urimeee urimeee merged commit 731c252 into main Jan 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant