feat: 이메일로 로그인으로 변경 #68
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,60 +1,65 @@ | ||
| // "use client"; | ||
| // import React from "react"; | ||
| // import Link from "next/link"; | ||
| // import { KakaoLoginButton } from "./SocialButtonList"; | ||
| // // import { KakaoLoginButton, GoogleLoginButton } from "./SocialButtonList"; | ||
| // import { | ||
| // Drawer, | ||
| // DrawerContent, | ||
| // DrawerTitle, | ||
| // DrawerTrigger, | ||
| // } from "@/components/ui/drawer"; | ||
| // | ||
| // interface LoginMethodDrawerProps { | ||
| // children: React.ReactNode; | ||
| // onKakaoLogin: () => void; | ||
| // onGoogleLogin: () => void; | ||
| // } | ||
| // | ||
| // export default function LoginMethodDrawer({ | ||
| // children, | ||
| // onKakaoLogin, | ||
| // onGoogleLogin, | ||
| // }: LoginMethodDrawerProps) { | ||
| // return ( | ||
| // <Drawer> | ||
| // <div className="typo-14-600 text-color-text-caption2 flex flex-col items-center leading-[1.6]"> | ||
| // <span>또는</span> | ||
| // <DrawerTrigger asChild>{children}</DrawerTrigger> | ||
| // </div> | ||
| // <DrawerContent | ||
| // showHandle={false} | ||
| // className="mx-auto flex h-auto min-h-[44.33dvh] w-full flex-col items-center px-4 pt-6 pb-[32px] md:max-w-[430px]" | ||
| // > | ||
| // <DrawerTitle className="sr-only">다른 로그인 방법</DrawerTitle> | ||
| // <div className="flex w-full flex-col items-start gap-2"> | ||
| // <span className="typo-20-700 text-bottomsheet-text-title"> | ||
| // 로그인/회원가입 | ||
| // </span> | ||
| // <span className="typo-14-500 text-[#999]"> | ||
| // 로그인과 회원가입 수단은 동일합니다. | ||
| // <br /> | ||
| // 원하는 계정으로 시작하세요. | ||
| // </span> | ||
| // </div> | ||
| // <KakaoLoginButton onClick={onKakaoLogin} className="mt-8 mb-4"> | ||
| // 카카오로 시작하기 | ||
| // </KakaoLoginButton> | ||
| // {/* <GoogleLoginButton className="w-full" onClick={onGoogleLogin}> | ||
| // 구글로 시작하기 | ||
| // </GoogleLoginButton> */} | ||
| // <div className="typo-14-500 text-bottomsheet-text-caption mt-6 flex flex-col items-center"> | ||
| // <span>혹은</span> | ||
| // <Link href="/login" className="all-[unset] cursor-pointer underline"> | ||
| // 이메일로 로그인 | ||
| // </Link> | ||
| // </div> | ||
| // </DrawerContent> | ||
| // </Drawer> | ||
| // ); | ||
| // } | ||
|
|
||
| // TypeScript 빌더 경고 우회용 Dummy Export | ||
| export default function DummyLoginMethodDrawer() { | ||
| return null; | ||
| } | ||
|
Comment on lines
+1
to
65
Contributor
There was a problem hiding this comment. 사용하지 않는 코드를 대량으로 주석 처리하여 방치하고, 빌드 경고를 피하기 위해 더미 컴포넌트를 export하는 것은 코드베이스의 가독성과 유지보수성을 저해합니다. Git과 같은 버전 관리 시스템을 통해 코드 이력이 보존되므로, 불필요해진 파일과 코드는 과감히 삭제하여 프로젝트 구조를 직관적으로 유지하는 것을 권장합니다. References
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'이메일로 로그인'과 같은 사용자 노출 문자열이 하드코딩되어 있습니다. 프로젝트의 유지보수성과 향후 다국어 지원(i18n) 가능성을 고려하여, 이러한 문자열은 별도의 로케일 파일이나 상수 관리 파일을 통해 관리하는 것이 좋습니다.