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/home page #57

Merged
merged 18 commits into from
Jun 25, 2024
Merged

Feat/home page #57

merged 18 commits into from
Jun 25, 2024

Conversation

bbung95
Copy link
Member

@bbung95 bbung95 commented Jun 16, 2024

📖 관련 문서

validation
https://formik.org/docs/api/useFormik
https://github.com/jquense/yup

✍🏻 변경사항:

  • formik과 yup을 사용하여 form validation 추가했습니다! (hookform과 formik 중에 formik이 조금더 익숙한거 같아 formik으로 추가했는데 확인하시고 말씀해주세요!!)
  • 북마크 등록 API 연동
  • 이미지 업로드 API 연동 (axios 수정 필요)
  • 이미지 미리보기 추가
  • 라우터 가드 추가 (임시)
    • 라우터 가드 임시로 추가했습니다! 현재 렌더링이 된 후에 라우터 이동이 있어서 수정해야할거 같아요!

🔍 확인할 목록

  • : formik & yup 라이브러리 확인
  • : 북마크 등록 API 정상 동작 확인

@bbung95 bbung95 requested a review from hoongding June 16, 2024 12:56
@bbung95 bbung95 self-assigned this Jun 16, 2024
Copy link

vercel bot commented Jun 16, 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 25, 2024 4:59pm

@bbung95 bbung95 marked this pull request as draft June 16, 2024 12:56
Copy link

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

Copy link

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

Copy link

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

@bbung95 bbung95 added ✨ feat 구현, 개선 사항에 관련된 내용입니다. 🎨 style UI 관련 이슈입니다. 🙋🏻 review 서로 리뷰 요청을 할 때 필요한 라벨입니다. labels Jun 17, 2024
@bbung95 bbung95 marked this pull request as ready for review June 17, 2024 12:37
Copy link

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

<div
className={cn([
'h-fit w-fit overflow-hidden rounded-xl',
'group-hover:shadow-layer group-hover:-translate-y-8 transition-all duration-200',
Copy link
Contributor

Choose a reason for hiding this comment

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

p99;
세상에,,, group hover가 있군요,,,

<img
className='rounded-full'
src={faviconUrl}
alt=''
Copy link
Contributor

Choose a reason for hiding this comment

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

p3;
alt 값 넣어주기!
favicon

</picture>
<span className='body-md text-text truncate'>{siteName}</span>
<div className='absolute right-0 hidden items-center gap-12 bg-surface *:text-icon-minimal group-hover:flex'>
<button onClick={(e) => e.stopPropagation()}>
Copy link
Contributor

Choose a reason for hiding this comment

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

p3;
이쪽 나중에 handleClick으로 바꿔서 공통되는 코드 제거해도 좋을 것 같습니다~

Copy link
Member Author

Choose a reason for hiding this comment

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

넵! 좋습니다!

@@ -121,7 +129,7 @@ const SideBar = () => {
{isOpenSidebar && (
<div className='px-12 py-[5px] body-sm-bold text-secondary'>리소스</div>
)}
<Menu>
<Menu onClick={() => handleNewWindowLink('https://forms.gle/dPi5voXeF3Fh7jve9')}>
Copy link
Contributor

Choose a reason for hiding this comment

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

p3;
handleNewWindowLink내의 url constant value로 빼서 주입해도 좋을 것 같아요!

</div>
</label>
) : (
<div
className='cursor-pointer w-full h-full flex justify-center items-center'
onClick={() => handleDeleteFile(files[0].key)}
>
{files[0].name}
<img className='aspect-[300/180] object-cover' src={String(files[0].src)} alt='' />
Copy link
Contributor

Choose a reason for hiding this comment

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

p3;
여기도 alt 추가하면 좋을 것 같습니당!

Copy link

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

@bbung95 bbung95 merged commit b4a9b02 into main Jun 25, 2024
4 of 5 checks passed
@hoongding hoongding deleted the feat/home-page branch July 13, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 구현, 개선 사항에 관련된 내용입니다. 🙋🏻 review 서로 리뷰 요청을 할 때 필요한 라벨입니다. 🎨 style UI 관련 이슈입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants