Skip to content

Conversation

@S-Gihun
Copy link

@S-Gihun S-Gihun commented May 15, 2025

โœ๏ธ ์ž‘์—… ๋‚ด์šฉ

#๏ธโƒฃ ์—ฐ๊ด€๋œ ์ด์Šˆ

#36


๐Ÿ“ท ์ž‘์—… ๊ฒฐ๊ณผ

์ž‘์—… ๊ฒฐ๊ณผ ์‚ฌ์ง„์„ ์—…๋กœ๋“œํ•ด์ฃผ์„ธ์š”.

2025-05-15.10.28.47.mov
2025-05-15.10.34.56.mov

๐Ÿ’ก ํ•จ๊ป˜ ๊ณต์œ ํ•˜๊ณ  ์‹ถ์€ ๋ถ€๋ถ„

ํ•ด๋‹น ์ฃผ์ฐจ๋ฅผ ๊ณต๋ถ€ํ•˜๋ฉด์„œ ํ•จ๊ป˜ ์ด์•ผ๊ธฐํ•˜๊ณ  ์‹ถ์€ ์ฃผ์ œ๋ฅผ ๋‚จ๊ฒจ์ฃผ์„ธ์š”.

(์–ด๋ ค์› ๋˜ ๋ถ€๋ถ„๊ณผ ํ•ด๊ฒฐ ๊ณผ์ •, ํ•ต์‹ฌ ์ฝ”๋“œ, ์ฐธ๊ณ ํ•œ ์ž๋ฃŒ ๋“ฑ)
์–‘์ด ๋„ˆ๋ฌด ๋งŽ๋„ค์š”.. ํŠธ๋Ÿฌ๋ธ” ์ŠˆํŒ…์€ Notion์— ์ •๋ฆฌ ํ•ด๋‘์—ˆ์Šต๋‹ˆ๋‹ค !

๊ทธ๋ฆฌ๊ณ  ์›น ์˜คํ”ˆ์ฑ„ํŒ…๋ฐฉ์—๋„ ์งˆ๋ฌธ ๋‚จ๊ฒจ๋‘์—ˆ์–ด์š”, ๋งคํŠœ๊ฐ€ ์ง์ ‘ ๋‹ต๋ณ€๊นŒ์ง€ ๋‹ฌ์•„์ฃผ์—ˆ์œผ๋‹ˆ ํ•œ ๋ฒˆ์”ฉ ๋ณด๋ฉด ์ข‹์„ ๊ฑฐ ๊ฐ™์•„์š”


๐Ÿค” ์งˆ๋ฌธ

ํ•ด๋‹น ์ฃผ์ฐจ ์›Œํฌ๋ถ์„ ๊ณต๋ถ€ํ•˜๋ฉด์„œ ๊ถ๊ธˆํ–ˆ๋˜ ์งˆ๋ฌธ๋“ค์„ ๋‚จ๊ฒจ์ฃผ์„ธ์š”.


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

  • ๋ชจ๋“  ํ•ต์‹ฌ ํ‚ค์›Œ๋“œ ์ •๋ฆฌ๋ฅผ ๋งˆ์ณค๋‚˜์š”?
  • ํ•ต์‹ฌ ํ‚ค์›Œ๋“œ์— ๋Œ€ํ•ด ์™„๋ฒฝํžˆ ์ดํ•ดํ•˜์˜€๋‚˜์š”?
  • ์‹ค์Šต/๋ฏธ์…˜์„ ์ˆ˜ํ–‰ํ•˜์˜€๋‚˜์š”?

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

  • pr ์ œ๋ชฉ์„ ์ปจ๋ฒค์…˜์— ๋งž๊ฒŒ ์ž‘์„ฑํ•˜์˜€๋‚˜์š”?
  • pr์— ํ•ด๋‹น๋˜๋Š” ์ด์Šˆ๋ฅผ ์—ฐ๊ฒฐํ•˜์˜€๋‚˜์š”?
  • ์ ์ ˆํ•œ ๋ผ๋ฒจ์„ ์„ค์ •ํ•˜์˜€๋‚˜์š”?
  • ์ฝ”๋“œ๋ฆฌ๋ทฐ๋ฅผ ์š”์ฒญํ•˜๊ธฐ ์œ„ํ•ด reviewer๋ฅผ ๋“ฑ๋กํ•˜์˜€๋‚˜์š”?
  • ๋‹‰๋„ค์ž„/main ๋ธŒ๋žœ์น˜์˜ ์ตœ์‹  ์ƒํƒœ๋ฅผ ๋ฐ˜์˜ํ•˜๊ณ  ์žˆ๋‚˜์š”?

@S-Gihun S-Gihun self-assigned this May 15, 2025
@S-Gihun S-Gihun added the ๐Ÿ’ก Mission ๋ฏธ์…˜ ์ˆ˜ํ–‰ label May 15, 2025
onSuccess: () => {
queryClient.invalidateQueries({
queryKey: [QUERY_KEY.comment],
exact: false,
Copy link
Contributor

Choose a reason for hiding this comment

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

QUERY_KEY.comment ํ‚ค๋งŒ ์‚ฌ์šฉ ์ค‘์ธ๊ฒƒ ๊ฐ™์€๋ฐ ์—ฌ๊ธฐ์„œ exact: false๋ฅผ ๋„ฃ์€ ์ด์œ ๊ฐ€ ๊ถ๊ธˆํ•ฉ๋‹ˆ๋‹ค

Copy link
Author

Choose a reason for hiding this comment

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

๊ธฐ๋ณธ ์„ค์ •๊ฐ’์ด exact: false ๋กœ ์•Œ๊ณ  ์žˆ์Šต๋‹ˆ๋‹ค.

๋‹ค๋ฅธ Optimistic Update ๊ตฌํ˜„ํ•œ hook ๋“ค๊ณผ ๋ช…ํ™•ํžˆ ๊ตฌ๋ถ„์„ ๋ณด์ด๊ธฐ ์œ„ํ•ด ์ฝ”๋“œ๋กœ์จ ๋ณด์—ฌ์คฌ์Šต๋‹ˆ๋‹ค.

์‚ฌ์‹ค ์—†์–ด๋„ ๋˜๋Š” ์ฝ”๋“œ๊ฐ€ ๋งž์Šต๋‹ˆ๋‹ค.

@S-Gihun S-Gihun merged commit 3938a49 into jett/main May 19, 2025
@S-Gihun S-Gihun deleted the jett/#36 branch June 20, 2025 06:38
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.

6 participants