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

Feat: Login 기능 구현 #44

Merged
merged 16 commits into from
Jun 1, 2024
Merged

Feat: Login 기능 구현 #44

merged 16 commits into from
Jun 1, 2024

Conversation

hoongding
Copy link
Contributor

📖 관련 문서

#39 closed

✍🏻 변경사항:

  • login 기능 구현
  • /login 페이지
  • /oauth2/redirect 페이지
    • token 파싱 후 메인페이지로 이동

🔍 확인할 목록

  • local 환경에서 동작하는지 확인.
  • 개발자 Tool 열어서 cookie에 accessToken 제대로 들어오는지 확인

@hoongding hoongding requested a review from rai-95 May 28, 2024 07:14
@hoongding hoongding self-assigned this May 28, 2024
Copy link

vercel bot commented May 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
packit-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 1, 2024 1:11am

@hoongding hoongding changed the title Feat/login Feat: Login 기능 구현 May 28, 2024
Copy link

🚀 Storybook 자동 배포 URL: https://6637bd73cba78215aa6e39fa-luyrzyvhtd.chromatic.com/

Copy link

🚀 Storybook 자동 배포 URL: https://6637bd73cba78215aa6e39fa-mlgxwcivba.chromatic.com/

Copy link
Contributor

@rai-95 rai-95 left a comment

Choose a reason for hiding this comment

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

로그인 고생하셨습니다ㅠㅠ!!

export const GOOGLE_LOGIN_URL = process.env.NEXT_PUBLIC_GOOGLE_LOGIN_BASE_URL;

env가 없어서 로그인 테스트를 못해봤습니다ㅠㅠ

return (
<div className='w-full h-full flex items-center justify-center flex-col'>
<div className='flex flex-col gap-32 justify-center items-center w-360 mb-[26px]'>
<Logo type='symbol' />
Copy link
Contributor

Choose a reason for hiding this comment

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

피그마에 full_logo로 되어 있는데 혹시 symbol로 해놓으신 이유가 있으실까요!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

login 피그마 나오기 전에 진행해서 제 마음대로 했었습니다!
ㅋㅋㅋ 바꿔놓을게요~

<br />
나만의 인사이트를 관리하세요
</span>
<button
Copy link
Contributor

Choose a reason for hiding this comment

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

Button 컴포넌트에 type=outline과 같은 ui인거 같아서 공통 컴포넌트로 변경해도 괜찮을거 같아요!


return (
<div className='w-full h-full flex items-center justify-center flex-col'>
<div className='flex flex-col gap-32 justify-center items-center w-360 mb-[26px]'>
Copy link
Contributor

Choose a reason for hiding this comment

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

w-360이 반영이 안되는거 같습니다!

반응형까지 생각했을때 모바일 사이즈에서 ~375px에서 양옆에 padding이 없이 width 100%로 적용되어 있어서 기본 padding 추가하면 좋을거 같아요!

@@ -0,0 +1,9 @@
<svg width="209" height="50" viewBox="0 0 209 50" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link
Contributor

Choose a reason for hiding this comment

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

width랑 height가 적용되서 size가 적용이 잘 안되는거 같아요!

그리고 Icon 사용할때 width, height로 사용하시나요!! 저는 className으로 적용하는거 같아서요 통일하면 좋을거 같습니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

width, height 삭제하는게 맞습니다!

body {
color: rgb(var(--foreground-rgb));
background: (var(--background-start-rgb));
@layer base {
Copy link
Contributor

Choose a reason for hiding this comment

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

폰트 추가 좋습니다!!

@@ -0,0 +1,26 @@
import _ from 'lodash';
Copy link
Contributor

Choose a reason for hiding this comment

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

오호 신기한 코드네요! 혹시 어떤 동작을 하는지 알 수 있을까요!??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이쪽에서는 api end point들을 모아놓으려고 합니다!!

@@ -0,0 +1,11 @@
import getConfig from 'next/config';

export const getEnv = (envName: string, fallback: string = ''): string => {
Copy link
Contributor

Choose a reason for hiding this comment

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

이 유틸은 env가 있을경우에는 env를 리턴하고 없을 경우에 fallback을 리턴하는걸까요!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

맞습니다!

Copy link

🚀 Storybook 자동 배포 URL: https://6637bd73cba78215aa6e39fa-tydxwkwizs.chromatic.com/

Copy link

github-actions bot commented Jun 1, 2024

🚀 Storybook 자동 배포 URL: https://6637bd73cba78215aa6e39fa-joudauvzdj.chromatic.com/

@hoongding hoongding merged commit fda4974 into main Jun 1, 2024
4 of 5 checks passed
Copy link

github-actions bot commented Jun 1, 2024

🚀 Storybook 자동 배포 URL: https://6637bd73cba78215aa6e39fa-sochypsmce.chromatic.com/

@hoongding hoongding deleted the feat/login branch July 13, 2024 11:22
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.

3 participants