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

Mission7/박건규 #44

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

geongyu09
Copy link
Collaborator

1. 구현 모습

May-15-2024 21-31-52
동영상을 클릭하면 사이트로 이동합니다!

2. 해결 과정

  • 모달을 띄워보았습니다.
    • react의 createPotal을 사용하였고, 앞으로 다른 다양한 모달을 띄울 수 있도록 createPotal로직을 따로 빼두었습니다.
    • 이를 통해, 만약 앞으로 다양한 모달을 띄워야 할 경우 ModalCreator() 컴포넌트에 props로 전달하는 방식 등으로 구현할 수 있을 것이라 기대하였습니다!
  • 모달의 경우, 사용자가 어느 날짜, 어느 시간대를 클릭하였는지 등의 정보값이 필요하였습니다. 이를 단순히 컴포넌트에서 props로 전달하게 되면 (컴포넌트 -> createPotal -> modal) 불필요한 의존 관계가 생긴다고 생각하였습니다. 이를 해결하기 위해 react의 context api 를 사용하여 결합력을 낮추도록 하였습니다. 즉 기존의 페이지에 보이는 컴포넌트에서 만들어진 정보를 모달 컴포넌트로 전달하는데 context 를 사용하였습니다.
  • 전역 상태 및 모달 등은 타 라이브러리를 사용하는 것도 괜찮아 보였지만, 이번 미션을 구현함에는 너무 투머치가 될 수 있을 것 같다는 생각이 들어 react만을 사용하게 되었습니다.

구현하고 보니.. 요구사항을 모두 지키지는 못하였네요..


3. To 리뷰어에게

  • 전체적으로 코드에 고칠점 및 부족한 점이 있을까요..?
  • [질문] 현업에서 정말 많은 곳에서 코드리뷰를 진행한다고 알고 있습니다. 저의 경우, 다른 사람의 코드를 잘 읽고 이해하지 못하는 경항이 있는데요. 혹시 다른 사람의 코드를 잘 읽을 수 있는 방법이나, 코드리뷰를 진행할 때 팁 같은게 있는지 궁금합니다!
  • [질문] 최근에 여러 과제 및 프로젝트를 하면서 AI의 도움을 많이 받고 있습니다. 이전에 GPT가 나오기 전과 후의 학부생 시절을 경험해보니, AI가 일의 효율을 정말로 높혀준다는 것을 느낄 수 있었습니다. 다만 정보 유출 등의 우려점도 있기도 합니다.. 그렇다면 현업에서는 AI를 사용하는지, 사용한다면 어떻게 사용하는지, 어떤 AI를 사용하는지 궁금합니다!

@geongyu09 geongyu09 requested a review from vvvpiano May 15, 2024 12:36
Copy link

vercel bot commented May 15, 2024

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

Name Status Preview Comments Updated (UTC)
donut-study ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2024 0:36am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mission 미션 입니다!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant