Skip to content

๐Ÿ”€ :: (#770) device token ์กฐํšŒ ๋น„๋™๊ธฐ ์ˆœ์ฐจ์  ์ฒ˜๋ฆฌ#771

Merged
uson1004 merged 1 commit intodevelopfrom
refactor/770-๋””๋ฐ”์ด์Šค-ํ† ํฐ-์กฐํšŒ-์—๋Ÿฌ
Oct 28, 2025

Hidden character warning

The head ref may contain hidden characters: "refactor/770-\ub514\ubc14\uc774\uc2a4-\ud1a0\ud070-\uc870\ud68c-\uc5d0\ub7ec"
Merged

๐Ÿ”€ :: (#770) device token ์กฐํšŒ ๋น„๋™๊ธฐ ์ˆœ์ฐจ์  ์ฒ˜๋ฆฌ#771
uson1004 merged 1 commit intodevelopfrom
refactor/770-๋””๋ฐ”์ด์Šค-ํ† ํฐ-์กฐํšŒ-์—๋Ÿฌ

Conversation

@uson1004
Copy link
Member

๊ฐœ์š”

device token ๋กœ์ง์„ ๋ฆฌํŒฉํ† ๋ง ํ•˜์˜€์Šต๋‹ˆ๋‹ค

์ž‘์—…์‚ฌํ•ญ

  • device token ์กฐํšŒ ์‹œ addOnCompleteListener๋กœ ์ธํ•ด task๊ฐ€ ์ •์ƒ์ ์œผ๋กœ ์˜ค์ง€ ์•Š๋Š” ๊ฒƒ์„ ํŒŒ์•…ํ•จ
  • withContext ์ˆœ์ฐจ ์ฒ˜๋ฆฌ ๋ฐ ์ฝ”๋ฃจํ‹ด ์ ์šฉ

์ถ”๊ฐ€ ๋กœ ํ•  ๋ง

@uson1004 uson1004 requested a review from a team October 27, 2025 11:22
@uson1004 uson1004 self-assigned this Oct 27, 2025
@uson1004 uson1004 added the ๐Ÿ‘พbug ๋ฒ„๊ทธ๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ label Oct 27, 2025
@uson1004
Copy link
Member Author

uson1004 commented Oct 28, 2025

lint์— ๋ฌธ์ œ๊ฐ€ ์žˆ๋Š”๊ฑธ๋กœ ํŒŒ์•…๋˜์„œ ๋จธ์ง€ํ•œ ํ›„ ๊ณ ์น˜๊ฒ ์Šต๋‹ˆ๋‹ค

@uson1004 uson1004 merged commit cb9f023 into develop Oct 28, 2025
0 of 5 checks passed
@uson1004 uson1004 deleted the refactor/770-๋””๋ฐ”์ด์Šค-ํ† ํฐ-์กฐํšŒ-์—๋Ÿฌ branch October 28, 2025 02:14
@parkuiery
Copy link
Member

addOnCompleteListener๋กœ ์ธํ•ด ์ •์ƒ์ž‘๋™ ์•ˆํ•˜๋Š”๊ฑฐ ๋งž๋‚˜์š”??

@parkuiery
Copy link
Member

ํŒŒ์ด์–ด๋ฒ ์ด์Šค ๊ณต์‹๋ฌธ์„œ์—์„œ๋Š” ์ „ ์ฝ”๋“œ๋ฅผ ํ™œ์šฉํ•˜๊ณ  ์žˆ๋Š”๋ฐ addOnCompleteListener์—์„œ๋Š” ๋ฐ์ดํ„ฐ ์ „๋‹ฌ์ด ์•ˆ๋˜๋Š”๊ฑธ๊นŒ์š”?

@uson1004
Copy link
Member Author

addOnCompleteListener๋ผ๊ณ ๋งŒ ์ ์—ˆ๋Š”๋ฐ ์ฒ˜์Œ์— ๋ฌธ์ œ ํ™•์ธ ํ–ˆ์„ ๋•Œ ๋ฐ”๋กœ ๋ณด์˜€๋˜ ๊ฑด suspend ํ‚ค์›Œ๋“œ๊ฐ€ ํ™œ์„ฑํ™”๊ฐ€ ์•ˆ๋˜์—ˆ๋˜ ๊ฒƒ์ด์˜€์Šต๋‹ˆ๋‹ค ํŒŒ์ด์–ด๋ฒ ์ด์Šค ์˜ˆ์ œ์—์„œ๋Š” addOnCompleteListener๋ฅผ ์‚ฌ์šฉํ•˜๋ฉด์„œ ์ฝ”๋ฃจํ‹ด์„ ์‚ฌ์šฉํ•˜์ง€ ์•Š์•˜์ง€๋งŒ DMS ๋กœ์ง์—์„  ์ฝ”๋ฃจํ‹ด์„ ์‚ฌ์šฉํ•ด์„œ ์ฐจ์ด๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฑฐ ๊ฐ™์Šต๋‹ˆ๋‹ค ํ•œ ์ค„ ์š”์•ฝํ•˜๋ฉด addOnCompleteListener ๋ฉ”์„œ๋“œ๋Š” ์ด๋ฒคํŠธ๋งŒ ๋“ฑ๋กํ•˜๊ณ  ๋ฆฌํ„ดํ•ด์„œ ์ฝ”ํˆฌ๋ฆฐ ์ž‘์—…์„ ๊ธฐ๋‹ค๋ฆฌ์ง€ ์•Š๊ณ  ๋ฆฌํ„ดํ•ด์„œ ์ƒ๊ธด ๋ฌธ์ œ๋ผ๊ณ  ์ •์ •ํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค ์ƒ๊ฐํ•ด๋ณด๋‹ˆ ์ฝ”๋ฃจํ‹ด์„ ์‚ฌ์šฉํ•˜๋Š” ์ด์œ ๋ฅผ ํ™•์ธํ•ด๋ณด์ง„ ์•Š์•„์„œ ์ด๊ฒƒ๋„ ํ•œ ๋ฒˆ ๊ณ ๋ คํ•ด๋ณด๊ฒ ์Šต๋‹ˆ๋‹ค

ํŒŒ์ด์–ด๋ฒ ์ด์Šค ๊ณต์‹๋ฌธ์„œ์—์„œ๋Š” ์ „ ์ฝ”๋“œ๋ฅผ ํ™œ์šฉํ•˜๊ณ  ์žˆ๋Š”๋ฐ addOnCompleteListener์—์„œ๋Š” ๋ฐ์ดํ„ฐ ์ „๋‹ฌ์ด ์•ˆ๋˜๋Š”๊ฑธ๊นŒ์š”?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

๐Ÿ‘พbug ๋ฒ„๊ทธ๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ

Projects

None yet

Development

Successfully merging this pull request may close these issues.

๋””๋ฐ”์ด์Šค ํ† ํฐ ์กฐํšŒ ์—๋Ÿฌ

2 participants