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] 공홈 어드민 소개 페이지 퍼블리싱 #129

Merged
merged 9 commits into from
Oct 30, 2024

Conversation

lydiacho
Copy link
Member

@lydiacho lydiacho commented Oct 27, 2024

✨ 구현 기능 명세

  • 헤더
  • 핵심 가치
  • 파트별 커리큘럼
  • 임원진
2024-10-27.5.30.34.mov

✅ PR Point

리터럴리 껍데기 퍼블리싱입니다.
언석오빠 작업물과 중복되는 스타일들은 추후 공통으로 빼주기 위해 최대한 동일하게 사용했고,
그 외에 대부분 mds 활용이라 작업한 내용이 적습니다 😓😓 정말 가볍게 봐주셔도 될거예요..!

다른 분들처럼 입력값 state 관리 같은 것도 다 우선 생략했고,
이미지 input을 형태별로 커스텀 가능하게 공통으로 만들면 좋을 것 같아서 구현하려고 했으나,
먼저 작업한 주용오빠 탭에서 제 뷰와 동일한 직사각형 이미지 input을 만들어주었길래 굳이 두번 만들지 않고 DUMMY div로 영역만 잡아놨어요. 중복 작업이 불필요해보여서요!

그래서 우선 이 껍데기 퍼블리싱 먼저 머지하고, 주용오빠 부분도 머지하게 되면
제가 주용오빠 file input 컴포넌트 공통으로 빼고, 저도 쓸 수 있는 형태로 확장시키는 작업 바로 진행 후, 제 뷰에도 반영시켜놓겠습니다.

그 외에 chip 클릭에 따라 내부 콘텐츠 변하는 것도, 입력값 관리와 연관된 부분이라 다음 단계에서 진행할 예정입니다.

머지 직후 TODO

  • image input 공통 컴포넌트 분리 및 확장
  • react-hook-form 도입해서 입력 상태값 관리 -> 와 더불어 chip 선택에 따른 컴포넌트 전환
  • 엣지케이스 파악, react-hook-form 통한 에러 처리

혹시 해당 PR에서 고도화하면 좋을 것 같다고 생각하시는 작업이 있다면 말씀해주세요 달아주시는대로 바로 추가 작업 하겠습니다!

Copy link
Member

@eonseok-jeon eonseok-jeon left a comment

Choose a reason for hiding this comment

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

이쁘네용 고생하셨습니다 :)

Copy link

@wuzoo wuzoo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !! LGTM ~ 🚀🚀

@lydiacho lydiacho merged commit a83edc5 into dev Oct 30, 2024
2 checks passed
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.

3 participants