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

강원대 FE_김정윤 2주차 과제 Step1 #50

Open
wants to merge 16 commits into
base: wjdsbs
Choose a base branch
from

Conversation

wjdsbs
Copy link

@wjdsbs wjdsbs commented Jul 5, 2024

안녕하세요 멘토님! 이번 주도 쉽지 않았던 것 같습니다. 잘 부탁드립니다!!

궁금한 점

  1. 컴포넌트를 많이 사용하는 페이지의 컴포넌트들을 MainPage, ThemePage처럼 폴더를 만들어 담게 되었는데, 프로젝트가 커지는 경우 바람직하지 못한 방식일지 궁금합니다.

  2. Theme 페이지 구현에서 재사용성을 고려하여 Header 섹션을 만든다는 부분을 잘 이해하지 못하여 많이 해매고 GPT도움도 많이 받았습니다. 일단 지금은 Theme 객체 한 개를 themes에 담아 놓은 상태인데, 나중에 백엔드와 합치게 되면 서버에서 Key와 value를 받아 Theme 페이지를 각각 다르게 출력하게 되는건가요? 또 그렇게 한다면 현재 코드에서 큰 수정이 필요한 상태인지 궁금합니다.

@wjdsbs wjdsbs changed the title 강원대 FE_김정윤 2주차 과제 STEP1 강원대 FE_김정윤 2주차 과제 Step1 Jul 5, 2024
@yujo11 yujo11 self-requested a review July 7, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant