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

refactor: 생년월일 입력 - 변경된 디자인 적용 및 리팩터링 #233

Closed
wants to merge 22 commits into from

Conversation

Doeunnkimm
Copy link
Member

@Doeunnkimm Doeunnkimm commented Mar 26, 2024

🤔 해결하려는 문제가 무엇인가요?

  • 변경된 생년월일 입력 부분 디자인 반영 (figma)

🎉 어떻게 해결했나요?

1. member/new/birth : 디자인 반영 및 유효성 검증 추가

  • 사용자가 입력을 시작하면, 유효성 검증할 수 있도록

GIFMaker_me (1)

2. 목표추가/date : 디자인 반영 및 유효성 검증 부분 리팩터링

GIFMaker_me (2)

3. my/update : 디자인 반영

GIFMaker_me (3)

📚 Attachment (Option)

👇 pull 받으면서 조금 망가져서 commits에서 이 부분만 확인 부탁 🙏 (위에서 6개)
image

Copy link

vercel bot commented Mar 26, 2024

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

Name Status Preview Comments Updated (UTC)
amazing3-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2024 2:05pm

@Doeunnkimm Doeunnkimm changed the title refactor: 변경된 디자인 적용 및 리팩터링 refactor: 생년월일 입력 - 변경된 디자인 적용 및 리팩터링 Mar 26, 2024
* feat: 타임라인 조회 api 수정

* feat: api 응답값 변경에 따라 props 및 타입 수정

* feat: TimelineCardBody 컴포넌트 구현

* feat: 타임라인 조회 api 변경
Copy link
Collaborator

@hjy0951 hjy0951 left a comment

Choose a reason for hiding this comment

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

최고다 도은아..💯💯💯💯💯💯💯💯💯💯

내가 짠거 다시보는데 극혐이였구나.. 고생 많았어.. 욕좀 했겠구나...
나 만나면 정신차리게 때려줘 😭

const [formattedValue, setFormattedValue] = useState<string>(intitalValue);
const placeholder = maxLength === MAX_DATE_LENGTH_UNTIL_MONTH ? 'YYYY.MM' : 'YYYY.MM.DD';

const handleInputChange = (event: ChangeEvent<HTMLInputElement>) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

와 여기부터 함수 토나오네

미안하다 도은... 최고야...👍

Doeunnkimm and others added 15 commits March 29, 2024 13:05
* chore: BottomSheet에 px 추가

* chore: props명 수정 및 시간정책 유틸 함수 적용

* chore: Comment 컴포넌트 애니메이션 적용

* feat: 댓글 조회 기능 추가

* feat: 댓글 추가 기능 구현

* feat: 댓글 삭제 기능 구현

* chore: 시간 정책 함수 주석 추가

* chore: atom → atoms 로 이동된 파일 적용

* chore: ios에서 키패드 이슈 대응.............

* refactor: isMyGoalAtom 위치 이동

* feat: 추가된 isMyGoal 값 적용 및 일부 스타일 수정

* chore: optimistic update하는 값 수정

* chore: GoalDetailContent에 aside 추가

* feat: 새로운 댓글이 있는지 조회하는 기능 추가

* chore: 변경 사항에 맞게 반영

* feat: 타임라인 조회 api 반영 (#231)

* feat: 타임라인 조회 api 수정

* feat: api 응답값 변경에 따라 props 및 타입 수정

* feat: TimelineCardBody 컴포넌트 구현

* feat: 타임라인 조회 api 변경

* chore: BottomSheet에 px 추가

* chore: props명 수정 및 시간정책 유틸 함수 적용

* chore: Comment 컴포넌트 애니메이션 적용

* feat: 댓글 조회 기능 추가

* feat: 댓글 추가 기능 구현

* feat: 댓글 삭제 기능 구현

* chore: 시간 정책 함수 주석 추가

* chore: atom → atoms 로 이동된 파일 적용

* chore: ios에서 키패드 이슈 대응.............

* refactor: isMyGoalAtom 위치 이동

* feat: 추가된 isMyGoal 값 적용 및 일부 스타일 수정

* chore: optimistic update하는 값 수정

* chore: GoalDetailContent에 aside 추가

* feat: 새로운 댓글이 있는지 조회하는 기능 추가

* chore: 변경 사항에 맞게 반영

---------

Co-authored-by: minkyung <newminkyung@gmail.com>
* chore: 정해진 status code에 대해서는 StandardErrorPage로 분리

* chore: cSpell.words 설정값 추가

* chore: ErrorPageLayout 컴포넌트 추가

* chore: status code를 보여줘야하는 경우에 사용할 컴포넌트 추가

* chore: 전역적으로 status code로 정해져있는 오류 화면 컴포넌트 추가

* chore: 네이밍 피드백 반영
* feat: 응원한 사람 조회 api 응답 값 수정 반영

* feat: 응원한 사람 조회 api 응답 값 수정 반영

* refactor: 응원 실패 시, early return

* fix: 응원하기 성공했을 때에도 에러 토스트가 뜨는 버그 수정
* feat: 피드 이모지 추가/삭제시 낙관적 업데이트 적용

* refactor: 불필요한 코드 삭제

* feat: 피드페이지에 다시 접근시 캐싱 데이터를 불러오지 않도록 언마운트될 때 캐시 내용 삭제
* 레이아웃 버그 수정 및 한글 문법 수정

* Textarea atom 버그 수정

* React Hook Form을 사용하는 TextField molecule 생성

* GoalUpdateForm 추가

* 불필요한 코드 삭제 및 RHFTextField.tsx에 Controller 반영

* RHFTextField를 textarea, input 태그 둘 다 핸들링 할 수 있게 수정

* RHFTextField 함수를 더 읽기 좋게 수정

* RHFDateField 생성

* RHFImagePicker와 RHFTagPicker 생성

* RHFTextField 로직 분리

* GoalUpdateForm 로직 완성

* 테스트하기 위해 추가된 promise 제거

* fix build error
Copy link

github-actions bot commented Apr 2, 2024

Copy link
Member

@newminkyung newminkyung left a comment

Choose a reason for hiding this comment

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

리베이스 한번만 부탁혀..

@Doeunnkimm
Copy link
Member Author

@minkyung00 리베이스하면 할 수록 망가지는 이유가 뭘까..? 깃이놈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants