로그인(Oauth) 페이지 레이아웃 완성 #55
Merged
Conversation
This file contains 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
rhehfl
requested review from
bluetree7878,
dmsdnWkd1234,
chamjin,
zzzRYT and
dg1418
as code owners
zzzRYT
reviewed
Oct 20, 2024
Comment on lines
+5
to
+9
| const kakaoURL = `https://kauth.kakao.com/oauth/authorize?client_id=${ | ||
| import.meta.env.VITE_KAKAO_REST_API_KEY | ||
| }&redirect_uri=${import.meta.env.VITE_KAKAO_REDIRECT_URL}&response_type=code`; | ||
| return ( | ||
| <> |
There was a problem hiding this comment.
oauth할 때 형식이 다 동일해서 util함수로 만들어서 관리하면 좋을 것 같습니다.
There was a problem hiding this comment.
넵 컴포넌트에 저런식으로 나열해놓는건 별로인거같긴 합니다 개인적으로는 api key와 redirect url 모두 고정적인 자원이니 카카오 url 전체를 그냥 env로 관리해도 괜찮을 것 같습니다
|
참고로 OAuth사용 시 이미지에 대한 가리드라인을 준수해여 합니다. (그거 준수안하면 OAuth못씀)크기에 따라서 다운받은 이미지는 깨질 수 있으니. 가이드라인에 맞게 디자이너분에게 디자인을 요청하거나, 외부에 존재하는 이미지 큰 배율로 받아와서 사용하면 좋을 것 같습니다. |
|
|
Closed
2 tasks
bluetree7878
approved these changes
Oct 21, 2024
| @@ -14,6 +15,7 @@ export default function Router() { | |||
| <Route path="/quest" element={<Quest />}></Route> | |||
| <Route path="/ranking" element={<Ranking />}></Route> | |||
| <Route path="/quiz/:section/:part" element={<Quiz />}></Route> | |||
| <Route path="/login" element={<Login />}></Route> | |||
| {/*어드민 페이지 부분 문제조회/추가 이외에 규모 확장 시 레포 분리 */} | |||
| <Route path="/admin" element={<Admin />} /> | |||
| <Route path="/admin/create-quiz" element={<CreateQuiz />}></Route> | |||
There was a problem hiding this comment.
이 부분 아까 같이 말씀 나눠봤지만 대문자든 소문자든 index든 해서 맞춰주면 좋을 것 같네용
There was a problem hiding this comment.
확인했습니다 폴더명이 Quiz, login등등으로 코드 컨밴션을 정할때 같이 정했어야 했는데 이런 부분을 체크하지 못한 것 같습니다. 이 부분은 따로 노션에 정리 후 이슈 생성해서 고치도록 하겠습니다
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔗 관련 이슈
54
📝작업 내용
간단하게 소셜로그인 화면 레이아웃을 생성했습니다
각 디벨로퍼 사이트에서 이런식으로 이미지파일을 제공해줘서 나중에 s3에 업로드 후 각 로그인버튼에 넣으면 좋을 것 같습니다!
링크는 일단 우선순위가 높은 카카오 소셜로그인 링크를 달아놓았습니다. (아직 인증을 못받아서 에러가 뜨긴 합니다)
🔍 변경 사항
💬리뷰 요구사항 (선택사항)