Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/217 startForegroundService-notification #218

Merged
merged 6 commits into from
Sep 1, 2023

Conversation

unam98
Copy link
Contributor

@unam98 unam98 commented Sep 1, 2023

๐Ÿ“Œ ๊ฐœ์š”

#217 startForegroundService-notification

โœจ ์ž‘์—… ๋‚ด์šฉ

  • timer๊ฐ€ ์„œ๋น„์Šค์—์„œ ๊ณ„์† ๋Œ์•„๊ฐ€๊ณ  ์žˆ๋‹ค๋Š” ๊ฒƒ์„ ์‚ฌ์šฉ์ž์—๊ฒŒ ๊ฐ€์‹œ์ ์œผ๋กœ ์•Œ๋ฆฌ๊ธฐ ์œ„ํ•ด (๊นœ๋นกํ•˜์ง€ ๋ง๋ผ๊ณ ) startForegroundService๋ฅผ ํ™œ์šฉํ•˜์—ฌ notification์„ ๋„์›Œ์ฃผ์—ˆ๊ณ  ์‹ค์‹œ๊ฐ„ timer๋„ ํ•จ๊ป˜ ๋ณด์—ฌ์ง€๊ฒŒ ํ•˜์˜€์Œ
  • ๋Ÿฌ๋‹์ด ์‹œ์ž‘๋  ๋•Œ ์Œ์„ฑ ์•ˆ๋‚ด๊ฐ€ ๋‚˜์˜ค๊ฒŒ ๊ตฌํ˜„ํ•ด์ฃผ์—ˆ์Œ (์˜์ƒ ์žฌ์ƒํ•  ๋•Œ ์†Œ๋ฆฌ์ผœ๋ณด์„ธ์š”!)

โœจ ๊ณ ๋ฏผ๋๋˜ ๋‚ด์šฉ

์Œ์„ฑ ์•ˆ๋‚ด ๊ธฐ๋Šฅ ๊ตฌํ˜„ํ•  ๋•Œ notification์— setSound๋ฅผ ํ™œ์šฉํ•˜๋ ค๊ณ  ํ–ˆ๋Š”๋ฐ ์•„๋ฌด๋ฆฌ ํ•ด๋„ ์†Œ๋ฆฌ๊ฐ€ ์•ˆ ๋‚˜์™€์„œ ๋‹ค๋ฅธ ๋ฐฉ๋ฒ•์„ ์ฐพ์•˜๊ณ  ๊ทธ๊ฒŒ ์ง€๊ธˆ pr์— ์˜ฌ๋ผ๊ฐ€๋Š” ๋‚ด์šฉ์ž…๋‹ˆ๋‹ค.

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท

KakaoTalk_20230902_060055404.mp4

์‹ค์‹œ๊ฐ„ timer data๋ฅผ notification ui์— ๋ฐ”์ธ๋”ฉํ•˜๋Š” ๊ฒƒ์€ ์•„์ง ๊ตฌํ˜„ ๋ชปํ•จ
๋„ค์ด๋ฒ„ ํด๋กœ๋ฐ” ๋”๋น™ ํ† ์ŠคํŠธ ๋ฉ”์„ธ์ง€๋กœ ์ถœ์ฒ˜ ํ‘œ๊ธฐ ํ•ด์ฃผ์–ด์•ผ ํ•จ
2023-09-02 05:49:24.347  2693-2693  NotificationService     system_server                        E  Muting recently noisy 0|com.runnect.runnect|1|null|10490
@unam98 unam98 merged commit 689454f into develop Sep 1, 2023
2 checks passed
@unam98 unam98 added the ์šฐ๋‚จ ๐Ÿผ ์šฐ๋‚จ ๋‹ด๋‹น label Sep 14, 2023
@unam98 unam98 self-assigned this Sep 30, 2023
@unam98 unam98 added the FIX ๐Ÿ’ฅ ๋ฒ„๊ทธ ๋ฐ ์˜ค๋ฅ˜ ํ•ด๊ฒฐ label Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIX ๐Ÿ’ฅ ๋ฒ„๊ทธ ๋ฐ ์˜ค๋ฅ˜ ํ•ด๊ฒฐ ์šฐ๋‚จ ๐Ÿผ ์šฐ๋‚จ ๋‹ด๋‹น
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant