Skip to content

Conversation

@sinji2102
Copy link
Member

@sinji2102 sinji2102 commented Feb 21, 2025

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

  • 회원가입 페이지 퍼블리싱
  • 로그인 페이지 퍼블리싱

📢 To Reviewers

  • 진짜 퍼블리싱만 하구 기능 연결 안 했습니다!! 연결할 기능들이 많지 않아서 API 연결할 때 한 번에 하면 될 것 같아용...~ 뭐더라? 아 구글 로그인........ 연결 오늘 새벽에 할게요 수민 오빠랑 하는 게 베스트같긴 한데 그렇게 하면 월요일 전에 연결할 수가 없을 듯
  • 여기서 쓰려구 Spacing 컴포넌트 만들었는데... 왜인지 잘 적용이 안 되네요 ㅠㅠ 일단 퍼블리싱 급해서 마진으로 다 처리했구 API 연결하고 조금 여유있거나 리팩토링... 이나 스프린트 하게 되면 적용할게요!

📸 스크린샷

  • 로그인
    image

  • 회원가입
    image

@sinji2102 sinji2102 linked an issue Feb 21, 2025 that may be closed by this pull request
3 tasks
@github-actions github-actions bot added 🚀 API API ✨ FEAT 기능 구현 labels Feb 21, 2025
@sinji2102 sinji2102 removed the 🚀 API API label Feb 21, 2025
Copy link
Collaborator

@daun-up daun-up left a comment

Choose a reason for hiding this comment

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

LGTM~


import MobileLayout from '@/components/mobile/layout';
import ImageLoginLogo from 'public/assets/images/image-login-logo.svg';
import IconGoogle from 'public/assets/icons/icon-google.svg';
Copy link
Collaborator

Choose a reason for hiding this comment

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

오! 바로 svg 컴포넌트로 선언 가능하네요? 15 에서 뭘 해도 안 됐던 거 같은데 굳~

@sinji2102 sinji2102 merged commit daa0354 into develop Feb 22, 2025
@sinji2102 sinji2102 deleted the feat/#28-login-publishing branch March 5, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 로그인/회원가입 페이지 퍼블리싱

3 participants