Skip to content

Conversation

@youdaeng2
Copy link
Member

๐Ÿ“ฆ Pull Request

๐Ÿ“ ์š”์•ฝ(Summary)

  • ๋งˆ์ดํ”„๋กœํ•„ ํŽ˜์ด์ง€(MyProfile)์— ๋กœ๊ทธ์ธ ๊ฐ€๋“œ ๋กœ์ง ์ ์šฉ

    • ๋น„๋กœ๊ทธ์ธ ์ƒํƒœ์—์„œ ์ ‘๊ทผ ์‹œ ErrorModal ํ‘œ์‹œ ํ›„ ๋กœ๊ทธ์ธ ํŽ˜์ด์ง€๋กœ ๋ฆฌ๋‹ค์ด๋ ‰ํŠธ
    • zustand์˜ isUserLoading ์ƒํƒœ๋ฅผ ํ™œ์šฉํ•ด ์‚ฌ์šฉ์ž ์ •๋ณด ๋กœ๋”ฉ ์™„๋ฃŒ ์—ฌ๋ถ€ ์ฒดํฌ
  • ๋กœ๋”ฉ ์ค‘์ผ ๋•Œ ์ „์ฒด ํ™”๋ฉด์— LoadingOverlay ํ‘œ์‹œ

    • ๊ธฐ์กด return <div>๋กœ๋”ฉ ์ค‘...</div> ๋ฐฉ์‹์„ ์ œ๊ฑฐํ•˜๊ณ , UI ๊นœ๋นก์ž„ ๋ฐฉ์ง€ ์ฒ˜๋ฆฌ
    • isUserLoading && <LoadingOverlay />๋กœ ์ž์—ฐ์Šค๋Ÿฝ๊ฒŒ ์ฒ˜๋ฆฌ

๐Ÿ’ฌ ๊ณต์œ ์‚ฌํ•ญ to ๋ฆฌ๋ทฐ์–ด

  • zustand์— isUserLoading ์ƒํƒœ๊ฐ€ ์žˆ์–ด์•ผ ํ•˜๊ณ , useInitUser()์—์„œ ์ด๋ฅผ ์ •ํ™•ํžˆ ์ฒ˜๋ฆฌํ•ด์•ผ ์ •์ƒ ์ž‘๋™ํ•ฉ๋‹ˆ๋‹ค.
  • ๋กœ๊ทธ์ธ ์—ฌ๋ถ€๊ฐ€ ํ™•์‹คํ•˜๊ฒŒ ์ฒดํฌ๋œ ๋’ค์— ํŽ˜์ด์ง€ ๋ Œ๋”๋ง๋จ
  • ๋น„๋กœ๊ทธ์ธ ์ ‘๊ทผ ์‹œ UX ํ๋ฆ„์ด ์ž์—ฐ์Šค๋Ÿฝ๊ฒŒ ์ „ํ™˜๋จ

๐Ÿ—‚๏ธ ๊ด€๋ จ ์ด์Šˆ

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท

๋น„๋กœ๊ทธ์ธ ์ƒํƒœ๋กœ ์ ‘๊ทผ
https://github.com/user-attachments/assets/69f4acde-de78-41b9-a362-f0e02401f21c

์ƒˆ๋กœ๊ณ ์นจ
https://github.com/user-attachments/assets/b6e5e71a-826e-4b06-b09d-511b95e14b5d

โœ… ์ฒดํฌ๋ฆฌ์ŠคํŠธ

  • ๋นŒ๋“œ ๋ฐ ํ…Œ์ŠคํŠธ ํ†ต๊ณผ
  • ESLint/Prettier ๊ฒ€์‚ฌ ํ†ต๊ณผ

@vercel
Copy link

vercel bot commented Aug 1, 2025

@youdaeng2 is attempting to deploy a commit to the 626-ju's projects Team on Vercel.

A member of the Team first needs to authorize it.

@youdaeng2 youdaeng2 merged commit 197a409 into codeit-part3-7:dev Aug 1, 2025
1 of 2 checks passed
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.

2 participants