Skip to content

Feat/158/comment edit modal#159

Merged
rak517 merged 1 commit intomainfrom
feat/158/comment-modify-modal
Feb 17, 2025
Merged

Feat/158/comment edit modal#159
rak517 merged 1 commit intomainfrom
feat/158/comment-modify-modal

Conversation

@rak517
Copy link
Collaborator

@rak517 rak517 commented Feb 16, 2025

โ“์ด์Šˆ

โœ๏ธ Description

  • ๋Œ“๊ธ€ ์ˆ˜์ • ํด๋ฆญ์‹œ ๋‚˜์˜ค๋Š” ๋ชจ๋‹ฌ ์ฐฝ์„ ๊ตฌํ˜„ํ•ฉ๋‹ˆ๋‹ค

แ„‰แ…ณแ„แ…ณแ„…แ…ตแ†ซแ„‰แ…ฃแ†บ 2025-02-16 18 03 12

โœ… Checklist

PR

  • Branch Convention ํ™•์ธ

    feat/* ๊ธฐ๋Šฅ ๊ตฌํ˜„, fix/* ๋ฒ„๊ทธ ์ˆ˜์ •, refactor/* ๊ฐœ์„ 

  • Base Branch ํ™•์ธ
  • ์ ์ ˆํ•œ Label ์ง€์ •
  • Assignee ๋ฐ Reviewer ์ง€์ •

Test

  • ๋กœ์ปฌ ์ž‘๋™ ํ™•์ธ

Additional Notes

  • (์—†์Œ)

@rak517 rak517 added Priority: Medium ๐Ÿ”– ์šฐ์„ ์ˆœ์œ„ ๋ณดํ†ต Type: Feature โœจ ์‹ ๊ทœ ๊ธฐ๋Šฅ labels Feb 16, 2025
@rak517 rak517 requested a review from SeokChan-Lee February 16, 2025 09:05
@rak517 rak517 self-assigned this Feb 16, 2025
@vercel
Copy link

vercel bot commented Feb 16, 2025

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Comments Updated (UTC)
taskify โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Feb 16, 2025 9:05am

@ToKyun02
Copy link
Owner

ToKyun02 commented Feb 17, 2025

๋Œ“๊ธ€ ์ˆ˜์ •๋„ ์ง€๋‚œ ๋ฒˆ์ฒ˜๋Ÿผ ๋ณ€ํ™”๊ฐ€ ์—†์œผ๋ฉด submit๋ชปํ•˜๊ฒŒ ๋ง‰์•„์ฃผ์‹œ๋ฉด ์ข‹์„ ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค!

์ถ”๊ฐ€๋กœ ๋Œ“๊ธ€์ด ๋งŽ์•„์ง€๋ฉด ๋Œ“๊ธ€ ๋ฆฌ์ŠคํŠธ ๊ธธ์ด ๋•Œ๋ฌธ์— ๋ชจ๋‹ฌ ๊ธธ์ด๊ฐ€ ๋„ˆ๋ฌด ๊ธธ์–ด์ง€๋Š” ๋ฌธ์ œ๊ฐ€ ์žˆ์Šต๋‹ˆ๋‹ค.

์ฝ”๋ฉ˜ํŠธ ๋ฆฌ์ŠคํŠธ ์˜์—ญ์— ์Šคํฌ๋กค ๋ถ€์—ฌํ•ด์„œ ๋ชจ๋‹ฌ ์„ธ๋กœ ๊ธธ์ด๋Š” ๋Œ“๊ธ€ ์ถ”๊ฐ€๋‚˜ ์‚ญ์ œ๋กœ ๋ณ€ํ•˜์ง€ ์•Š์•˜์œผ๋ฉด ์ข‹๊ฒ ์Šต๋‹ˆ๋‹ค!
์ œ๊ฐ€ ๊ตฌํ˜„ํ–ˆ๋˜ Assign Input์ฒ˜๋Ÿผ ๊ณ ์ •์ ์ธ ๋†’์ด๊ฐ€ ์ข‹์„ ๊ฒƒ ๊ฐ™์Šด๋‹ค.

๊ทธ๋ฆฌ๊ณ , isLoading, isFetchingNextPage ์ƒํƒœ๊ฐ€ ๋˜๋ฉด ๋กœ๋”ฉ ์Šคํ”ผ๋„ˆ๋‚˜ ์Šค์ผˆ๋ ˆํ†ค UI ๋„ฃ์œผ๋ฉด ์ข‹์„ ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค!

๊ธฐ๋Šฅ์ ์ธ ๊ตฌํ˜„์€ ํ˜„์žฌ PR์ด ๋งˆ์ง€๋ง‰์ด๋ผ์„œ ์ œ๊ฐ€ ๋‚˜์—ดํ•œ ๊ฐœ์„  ์‚ฌํ•ญ์€ ์ด์Šˆํ•˜๋‚˜ ํŒŒ์„œ ์ž‘์—…ํ•˜์‹ฌ ๋  ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค!

image

@rak517 rak517 merged commit 43e2391 into main Feb 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium ๐Ÿ”– ์šฐ์„ ์ˆœ์œ„ ๋ณดํ†ต Type: Feature โœจ ์‹ ๊ทœ ๊ธฐ๋Šฅ

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: ๋Œ“๊ธ€ ์ˆ˜์ • ๋ชจ๋‹ฌ ์ž‘์—…

3 participants