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

Feature/FE-022 - 회원가입 페이지 / 웰컴 페이지 #55

Merged
merged 7 commits into from
Jul 1, 2023

Conversation

jieunpark247
Copy link
Contributor

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 관련된 이슈와 연결 시켰나요?
  • Target Branch를 올바르게 설정했나요?
  • Label을 알맞게 설정했나요?

작업 내역

  • �회원가입 페이지 / 웰컴 페이지

비고

  • 정보 제공 동의 버튼 여러번 클릭시 잘 동작 안되는 이슈가 있습니다 -> 추후 수정 예정

@sonarcloud
Copy link

sonarcloud bot commented Jun 30, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 7 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@naro-Kim naro-Kim Jul 1, 2023

Choose a reason for hiding this comment

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

약관 동의 과정에서 누르지 않고 form을 submit한 경우, 동의 버튼이 disabled 되는 에러가 있는 것 같습니다.
추후 수정에 포함해야할 것 같습니다~

스크린샷 2023-07-01 오전 9 28 23

return (
<div className={wrapper}>
<Title textAlign="start">인증 메일을 보냈습니다.</Title>
<Typography as="p" variant="body2" className={description}>
Copy link
Contributor

Choose a reason for hiding this comment

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

현재는 @Samsung으로 주소가 표시되어있는데 '{${userInfo.email}@samsung.com} '로 표기해주어야 할 거 같습니다

@@ -0,0 +1,21 @@
import { keyframes, style } from '@vanilla-extract/css';

const fadeInAnimation = keyframes({
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

Choose a reason for hiding this comment

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

저도 추가로 constants 폴더에 관리해보도록 하겠습니다~ 굿굿

@jieunpark247 jieunpark247 merged commit 2436805 into dev Jul 1, 2023
3 checks passed
@jieunpark247 jieunpark247 deleted the feature/FE-022 branch July 1, 2023 00:40
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.

2 participants