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

[WC-59] feat : LoadingProvider 구현 및 적용 #34

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

Conversation

younoah
Copy link
Member

@younoah younoah commented Mar 14, 2022

🚅 PR 한 줄 요약

LoadingProvider 구현 및 적용

🧑‍💻 PR 세부 내용

  • 로딩 상태를 전역으로 다루도록 LoadingProvider를 구현하였습니다.
  • LoadingProvider 내부에 Spinner 컴포넌트가 존재하여 페이지에서는 단순하게 setLoading으로 로딩을 처리할 수 있습니다.
  • Spinner 컴포넌트와 로딩상태를 갖는 모든 페이지에서 LoadingProvider를 도입하여 변경하였습니다.

@netlify
Copy link

netlify bot commented Mar 14, 2022

✔️ Deploy Preview for wafflecard ready!

🔨 Explore the source changes: 863f8b8

🔍 Inspect the deploy log: https://app.netlify.com/sites/wafflecard/deploys/622f21d13d67f000080131a1

😎 Browse the preview: https://deploy-preview-34--wafflecard.netlify.app/

<Router />
</WaffleCardsProvider>
</ModalsProvider>
</LoadingProvider>
Copy link
Collaborator

Choose a reason for hiding this comment

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

그럼 Spinner컴포넌트는
</ModalsProvider>
여기에
</LoadingProvider>
항상 있는 거고 필요한 페이지에서 true, false해서 보여지고 안보여지게 하는 거 맞죠??

Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 리뷰해놓고 제출을 안하고있었네요 ㅠㅠㅋㅋ

Copy link
Member Author

Choose a reason for hiding this comment

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

@yyoooon
Spinner 컴포넌트 </LoadingProvider> 에서만 관리해여!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants