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: scaffold pages (Post 플로우) #14

Merged
merged 6 commits into from
Apr 21, 2024
Merged

Conversation

yoopark
Copy link
Contributor

@yoopark yoopark commented Apr 21, 2024

Summary

Post 플로우에 대한 뷰 스캐폴딩을 진행하였습니다.

  • PostPage
  • CommentsPage
  • ReportPostId, ReportCommentsId

#4 의 말미에 언급하였던 작업 진행 계획 중 3번째 작업을 진행 중입니다.

  1. shadcn/ui 컴포넌트 커스텀하기 - 피그마 시안 보고 필요한 variant만 남기는 방식으로 커스텀
  2. shadcn/ui에는 없지만 필요한 기본 컴포넌트 제작하기
  3. 피그마 시안에 따라 뷰 스캐폴딩
  4. 백엔드와 로직 연결

#9 에서 언급하였던 다음 PR 계획 중 2번째 작업에 해당합니다.

  1. 기획 및 디자인 부족한 부분 논의(2024. 04. 19. 예정)에 대한 결정 사항 반영
  2. Post 플로우 스캐폴딩 (게시글 보기, 댓글 보기, 게시글 및 댓글 신고하기)
  3. 뷰 스캐폴딩 과정 전반에 대한 코드 퀄리티 향상, 안정화
  4. 백엔드 모킹
  5. 백엔드와 로직 연결 (init: install @shadcn/ui and storybook #4 기준으로는 4번째 작업 시작)

Describe your changes

  • 라우팅 변경사항 없습니다.
  • components/ui 변경사항 1개 있습니다.
    • 9e2b97c Button 컴포넌트의 size: none 추가

답글 페이지를 아직 제작하지 않았습니다.

기타 변경사항은 다음과 같습니다.

  • 223d613 copyCurrentPage 로직이 use client가 필요한 관계로, asChild 방식을 이용한 리팩토링
  • cb7b108 댓글과 답글 관련 더미, 더미 생성 로직 추가

Issue number and link

@yoopark yoopark added the enhancement New feature or request label Apr 21, 2024
@yoopark yoopark self-assigned this Apr 21, 2024
Copy link

vercel bot commented Apr 21, 2024

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

Name Status Preview Updated (UTC)
team-sprint-blccu-frontend ✅ Ready (Inspect) Visit Preview Apr 21, 2024 3:51am

@yoopark yoopark merged commit 2750a90 into main Apr 21, 2024
4 of 5 checks passed
@yoopark yoopark deleted the feat/scaffold-pages-3 branch April 21, 2024 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant