Skip to content

๐Ÿ”€ :: (#259) - ์„ธํƒ ์™„๋ฃŒ ์•Œ๋ฆผ ์ƒํƒœ ๊ฐฑ์‹  ๋ฐ ์•Œ๋ฆผ ๋ชฉ๋ก ํŒŒ์‹ฑ ์ˆ˜์ • - #260

Merged
leesengjun merged 2 commits into
developfrom
fix/#259-laundry-notification-state-v2
Sep 1, 2026
Merged

๐Ÿ”€ :: (#259) - ์„ธํƒ ์™„๋ฃŒ ์•Œ๋ฆผ ์ƒํƒœ ๊ฐฑ์‹  ๋ฐ ์•Œ๋ฆผ ๋ชฉ๋ก ํŒŒ์‹ฑ ์ˆ˜์ •#260
leesengjun merged 2 commits into
developfrom
fix/#259-laundry-notification-state-v2

Conversation

@leesengjun

Copy link
Copy Markdown
Contributor

๐Ÿ’ก ๊ฐœ์š”

์„ธํƒ ์™„๋ฃŒ ์•Œ๋ฆผ ๊ด€๋ จ ์ƒํƒœ ๊ฐฑ์‹  ๋ฐ ์•Œ๋ฆผ ๋ชฉ๋ก ํŒŒ์‹ฑ ๋ฌธ์ œ๋ฅผ ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

RUNNING ๋˜๋Š” IN_USE ์ƒํƒœ์—์„œ expectedCompletionTime์ด ์—†๋Š” ๊ฒฝ์šฐ์—๋„
ํ•„์š”ํ•œ ์ƒํƒœ ๊ฐฑ์‹ ์ด ์ด์–ด์งˆ ์ˆ˜ ์žˆ๋„๋ก ์ฒ˜๋ฆฌํ•˜๊ณ ,
์•Œ๋ฆผ ๋ชฉ๋ก API ์‘๋‹ต์ด data wrapper ์•ˆ์— ๋‚ด๋ ค์˜ค๋Š” ๊ตฌ์กฐ๋ฅผ ์ •์ƒ์ ์œผ๋กœ ํŒŒ์‹ฑํ•˜๋„๋ก ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

๊ธฐ์กด ๋ธŒ๋žœ์น˜๊ฐ€ develop ๊ธฐ์ค€์œผ๋กœ ์ƒ์„ฑ๋˜์–ด main ๊ธฐ์ค€์œผ๋กœ ์ž๋™ ๋ณ‘ํ•ฉ์ด ์–ด๋ ค์šด ์ƒํƒœ์˜€๊ธฐ ๋•Œ๋ฌธ์—,
main ๊ธฐ์ค€ ์ƒˆ ๋ธŒ๋žœ์น˜์—์„œ #259 ์ž‘์—… ์ปค๋ฐ‹๋งŒ ๋ฐ˜์˜ํ•ด PR์„ ๋‹ค์‹œ ์ƒ์„ฑํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ”— ๊ด€๋ จ ์ด์Šˆ

Close #259

๐Ÿ“ƒ ์ž‘์—…๋‚ด์šฉ

  • ์„ธํƒ/๊ฑด์กฐ ์ƒํƒœ polling ์กฐ๊ฑด ์ˆ˜์ •
  • RUNNING / IN_USE ์ƒํƒœ์—์„œ expectedCompletionTime์ด ์—†๋Š” ๊ฒฝ์šฐ์—๋„ polling์ด ์œ ์ง€๋˜๋„๋ก ์ฒ˜๋ฆฌ
  • ์˜ˆ์ƒ ์ข…๋ฃŒ ์‹œ๊ฐ„์ด ๋‚˜์ค‘์— ์ฑ„์›Œ์ง€๋Š” ๊ฒฝ์šฐ ์•ฑ์—์„œ ๋‹ค์‹œ ๋ฐ›์•„์˜ฌ ์ˆ˜ ์žˆ๋„๋ก ์ˆ˜์ •
  • ์•Œ๋ฆผ ๋ชฉ๋ก API response wrapper ํŒŒ์‹ฑ ๋กœ์ง ์ˆ˜์ •
  • data.notifications ๊ตฌ์กฐ์—์„œ ์•Œ๋ฆผ ๋ชฉ๋ก์„ ์ •์ƒ์ ์œผ๋กœ ํŒŒ์‹ฑํ•˜๋„๋ก ์ˆ˜์ •
  • main ๊ธฐ์ค€ ์ƒˆ ๋ธŒ๋žœ์น˜์—์„œ ๐Ÿ› :: ์„ธํƒ ์™„๋ฃŒ ์•Œ๋ฆผ ์ƒํƒœ ๊ฐฑ์‹  ๋ฐ ์•Œ๋ฆผ ๋ชฉ๋ก ํŒŒ์‹ฑ ํ™•์ธย #259 ์ž‘์—… ์ปค๋ฐ‹๋งŒ ๋ฐ˜์˜

๐Ÿ” ํ…Œ์ŠคํŠธ ๋ฐฉ๋ฒ•

  • flutter analyze ์‹คํ–‰
  • ์„ธํƒ/๊ฑด์กฐ ์˜ˆ์•ฝ ํ˜„ํ™ฉ ํ™”๋ฉด ์ง„์ž… ํ™•์ธ
  • RUNNING / IN_USE ์ƒํƒœ์—์„œ ์˜ˆ์ƒ ์ข…๋ฃŒ ์‹œ๊ฐ„์ด ์—†๋Š” ๊ฒฝ์šฐ ์ƒํƒœ ๊ฐฑ์‹ ์ด ์œ ์ง€๋˜๋Š”์ง€ ํ™•์ธ
  • ์•Œ๋ฆผ ๋ชฉ๋ก API ์‘๋‹ต์ด ์ •์ƒ์ ์œผ๋กœ ํŒŒ์‹ฑ๋˜๋Š”์ง€ ํ™•์ธ
  • ๋ถˆํ•„์š”ํ•œ ๋ณ€๊ฒฝ ํŒŒ์ผ์ด ํฌํ•จ๋˜์ง€ ์•Š์•˜๋Š”์ง€ ํ™•์ธ

๐Ÿ–ผ๏ธ ์Šคํฌ๋ฆฐ์ƒท

์—†์Œ

๐Ÿ™‹โ€โ™‚๏ธ ์งˆ๋ฌธ์‚ฌํ•ญ

  • RUNNING / IN_USE ์ƒํƒœ์—์„œ polling์„ ์œ ์ง€ํ•˜๋Š” ๊ธฐ์ค€์ด ์ ์ ˆํ•œ์ง€ ํ™•์ธ ๋ถ€ํƒ๋“œ๋ฆฝ๋‹ˆ๋‹ค.
  • ์•Œ๋ฆผ ๋ชฉ๋ก API wrapper ํŒŒ์‹ฑ ๋ฐฉ์‹์ด ์„œ๋ฒ„ ์‘๋‹ต ๊ตฌ์กฐ์™€ ๋งž๋Š”์ง€ ํ™•์ธ ๋ถ€ํƒ๋“œ๋ฆฝ๋‹ˆ๋‹ค.
  • ๊ฐœ์„ ํ•  ์ , ์˜คํƒ€, ์ฝ”๋“œ์— ์ด์ƒํ•œ ๋ถ€๋ถ„์ด ์žˆ๋‹ค๋ฉด Comment ๋‹ฌ์•„์ฃผ์„ธ์š”.

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

  • ์ฝ”๋“œ๊ฐ€ ์ •์ƒ์ ์œผ๋กœ ์ปดํŒŒ์ผ๋˜๊ณ  ์‹คํ–‰๋˜๋Š”์ง€ ํ™•์ธํ–ˆ์Šต๋‹ˆ๋‹ค.
  • flutter analyze๋ฅผ ์‹คํ–‰ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ๋ถˆํ•„์š”ํ•œ ์ฝ”๋“œ๊ฐ€ ์—†๋Š”์ง€ ํ™•์ธํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ์ฝ”๋“œ ์Šคํƒ€์ผ ๊ฐ€์ด๋“œ๋ฅผ ์ค€์ˆ˜ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ๊ธฐ์กด ๊ธฐ๋Šฅ์ด ์ •์ƒ์ ์œผ๋กœ ์ž‘๋™ํ•˜๋Š”์ง€ ํ™•์ธํ–ˆ์Šต๋‹ˆ๋‹ค.

@leesengjun leesengjun self-assigned this Aug 31, 2026

@aiden30015 aiden30015 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

์ˆ˜๊ณ ํ•˜์…จ์Šต๋‹ˆ๋‹ค

@leesengjun
leesengjun changed the base branch from main to develop September 1, 2026 06:41
@leesengjun
leesengjun merged commit 1db0c8f into develop Sep 1, 2026
2 of 3 checks passed
@leesengjun
leesengjun deleted the fix/#259-laundry-notification-state-v2 branch September 1, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants