Skip to content

Comments

[fix/goal-243] ๐Ÿšจ Fix : ๋ชฉํ‘œ ์—…๋ฐ์ดํŠธ ์‹œ ์ฟผ๋ฆฌํ‚ค ์ถ”๊ฐ€#244

Merged
goodaseul merged 1 commit intodevfrom
fix/goal-243
Jan 1, 2026
Merged

[fix/goal-243] ๐Ÿšจ Fix : ๋ชฉํ‘œ ์—…๋ฐ์ดํŠธ ์‹œ ์ฟผ๋ฆฌํ‚ค ์ถ”๊ฐ€#244
goodaseul merged 1 commit intodevfrom
fix/goal-243

Conversation

@swallowedB
Copy link
Collaborator

์š”์•ฝ

  • ๋ณ€๊ฒฝ ๋ชฉ์ (์™œ?)

    • ๋ชฉํ‘œ ์ˆ˜์ • ํ›„ ์ƒ์„ธ ํŽ˜์ด์ง€๋Š” ์ •์ƒ์ ์œผ๋กœ ๊ฐฑ์‹ ๋˜์ง€๋งŒ, ๋„ค๋น„๊ฒŒ์ด์…˜์—์„œ ์‚ฌ์šฉํ•˜๋Š” goals ๋ฆฌ์ŠคํŠธ ์บ์‹œ๊ฐ€ ๊ฐฑ์‹ ๋˜์ง€ ์•Š์•„
      ์ˆ˜์ • ๋‚ด์šฉ์ด ์ฆ‰์‹œ ๋ฐ˜์˜๋˜์ง€ ์•Š๋Š” ๋ฌธ์ œ๋ฅผ ํ•ด๊ฒฐ
  • ์ฃผ์š” ๋ณ€๊ฒฝ(๋ฌด์—‡์„?)

    • ๋ชฉํ‘œ ์ˆ˜์ • mutation ์„ฑ๊ณต ์‹œ,
      goals ์ƒ์„ธ(detail)๋ฟ ์•„๋‹ˆ๋ผ ๋ฆฌ์ŠคํŠธ(list) ์ฟผ๋ฆฌ๋„ ํ•จ๊ป˜ ๊ฐฑ์‹ ํ•˜๋„๋ก ์ฒ˜๋ฆฌ

๋ณ€๊ฒฝ ๋‚ด์šฉ

  • ๋ชฉํ‘œ ์ˆ˜์ • mutation์—์„œ goalsQueryKeys.list() invalidate ์ถ”๊ฐ€
  • ์ƒ์„ธ(detail) ์ฟผ๋ฆฌ์™€ ๋ฆฌ์ŠคํŠธ(list) ๊ฐ„ ์บ์‹œ ๋™๊ธฐํ™” ๋ณด์žฅ
  • ๋„ค๋น„๊ฒŒ์ด์…˜ ๋ชฉํ‘œ ๋ชฉ๋ก์ด ์ˆ˜์ • ์งํ›„ ์ฆ‰์‹œ ๋ฐ˜์˜๋˜๋„๋ก ๊ฐœ์„ 

๋ฐฐ๊ฒฝ ์„ค๋ช…

๊ธฐ์กด ๊ตฌ์กฐ์—์„œ๋Š”:

  • ๋ชฉํ‘œ ์ˆ˜์ • ์‹œ goalsQueryKeys.detail(goalId)๋งŒ ๊ฐฑ์‹ 
  • ๋„ค๋น„๊ฒŒ์ด์…˜์€ goalsQueryKeys.list() ์ฟผ๋ฆฌ๋ฅผ ์‚ฌ์šฉํ•˜๊ณ  ์žˆ์–ด
    ์บ์‹œ ๋ถˆ์ผ์น˜๋กœ ์ธํ•ด UI๊ฐ€ ์—…๋ฐ์ดํŠธ๋˜์ง€ ์•Š๋Š” ๋ฌธ์ œ๊ฐ€ ๋ฐœ์ƒ

์ด๋ฒˆ ์ˆ˜์ •์œผ๋กœ:

  • ๋ชฉํ‘œ ์ˆ˜์ • ํ›„ ๋ฆฌ์ŠคํŠธ ์ฟผ๋ฆฌ๋„ ๋ฌดํšจํ™”(invalidate)ํ•˜์—ฌ
    ์„œ๋ฒ„ ๋ฐ์ดํ„ฐ์™€ UI ์ƒํƒœ๋ฅผ ์ผ๊ด€๋˜๊ฒŒ ์œ ์ง€
  • React Query ์บ์‹œ ๋‹จ์ผ ์†Œ์Šค ์›์น™์„ ๊ฐ•ํ™”

๊ด€๋ จ ์ด์Šˆ

close #243

@vercel
Copy link
Contributor

vercel bot commented Jan 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
slice Ready Ready Preview, Comment Jan 1, 2026 1:10pm

@goodaseul goodaseul merged commit 387cc37 into dev Jan 1, 2026
3 checks passed
@goodaseul goodaseul deleted the fix/goal-243 branch January 1, 2026 13:13
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.

๐Ÿšจ Fix : ๋ชฉํ‘œ๋ฆฌ์ŠคํŠธ ์—…๋ฐ์ดํŠธ ๋ฌธ์ œ

2 participants