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] 게스트 페이지에서 질문 게시판이 블러 처리된 상태로 보이도록 추가 #143

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

geongyu09
Copy link
Collaborator

📌 관련 이슈

closed #123

✨ PR 내용

게스트로 디테일 페이지에 가게 되었을 때 질문게시판이 블러 처리된 상태로 보이도록 하였습니다.

image

실제로 블러 처리된 데이터들은 아무런 의미 없는 Lorem Ipsum 문장들입니다.

성능을 향상시키기 위하여 해당 컴포넌트는 서버 컴포넌트로 하여 미리 렌더링된 페이지를 볼 수 있도록 하였습니다.

추가적으로 디테일 페이지의 일부 코드를 리팩토링하였습니다.

  • 불필요하게 section으로 묶여있던 section을 풀어주었습니다.
  • 컨밴션에 맞추어 section은 독립적으로 동작할 수 있도록 상위 컴포넌트외의 의존성(programId, accessType등)을 제거하고, section 내부에서 해당 값들을 가져와 사용할 수 있도록 하였습니다.

추가

추가적으로 컴포넌트 테스트 추가는 따로 pr을 작성하도록 하겠습니다

- 명세에 맞추어 page는 여러 독립적인 section으로 이루어지도록 구성하기 위해서 불필요하게 section 두개를 묶은 부분(ProgramHeader, ProgramDetail)을 따로 독립적인 section으로 수정
- 분리함에 따라서 기존에 두 컴포넌트를 묶어서 로더를 적용하는 부분 분리
dashboard에서 많은 책임을 가지고 있던 부분에서 탭 부분은 분리
@geongyu09 geongyu09 added the feature🔥 🔥기능 추가! 🔥 label Nov 30, 2024
@geongyu09 geongyu09 self-assigned this Nov 30, 2024
Copy link

vercel bot commented Nov 30, 2024

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

Name Status Preview Comments Updated (UTC)
eeos-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 30, 2024 7:59am
eeos-fe-develop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 30, 2024 7:59am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature🔥 🔥기능 추가! 🔥
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant