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

通知音などの発音方法を変え、iOSで音声出力が競合しないようにする #12339

Merged

Conversation

samunohito
Copy link
Member

@samunohito samunohito commented Nov 15, 2023

What

#12340 の対応です。
この対応により、iOSでバックグラウンド再生中の音楽がMisskeyの通知音などを受けて停止してしまう現象が改善されます。

Why

詳細は #12340 をご参照願います

Additional info (optional)

プルリクを出しているブランチをみすてむずに乗せてもらい、実動作確認済みです。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Nov 15, 2023
Copy link

codecov bot commented Nov 15, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ca81f0d) 78.84% compared to head (35e05ad) 78.74%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12339      +/-   ##
===========================================
- Coverage    78.84%   78.74%   -0.10%     
===========================================
  Files          946      176     -770     
  Lines       102153    23296   -78857     
  Branches      8255      495    -7760     
===========================================
- Hits         80544    18345   -62199     
+ Misses       21609     4951   -16658     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

github-actions bot commented Nov 15, 2023

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@samunohito samunohito changed the title [wip] HTMLAudioElementを使わないようにする [wip] 通知音などの発音方法を変え、iOSで音声出力が競合しないようにする Nov 15, 2023
@samunohito samunohito changed the title [wip] 通知音などの発音方法を変え、iOSで音声出力が競合しないようにする 通知音などの発音方法を変え、iOSで音声出力が競合しないようにする Nov 15, 2023
@samunohito samunohito marked this pull request as ready for review November 15, 2023 08:34
@syuilo syuilo merged commit 38d6580 into misskey-dev:develop Nov 15, 2023
17 of 18 checks passed
@syuilo
Copy link
Member

syuilo commented Nov 15, 2023

🙏🏻 🙏🏻 🙏🏻 🙏🏻

@syuilo syuilo mentioned this pull request Nov 15, 2023
@kakkokari-gtyih
Copy link
Contributor

Androidでも改善してます👍👍👍👍

@tai-cha
Copy link
Member

tai-cha commented Nov 15, 2023

Androidでも改善してるなら元issue閉じちゃって平気かしら

@kakkokari-gtyih
Copy link
Contributor

(WebAudio経由での発音になったおかげで、副作用で通知などへのサウンドのレスポンスも良くなってる)

@kakkokari-gtyih
Copy link
Contributor

ただ、複数の通知が同時に入った場合に多重で通知音が鳴って音割れしそう

n1lsqn pushed a commit to n1lsqn/misskey that referenced this pull request Nov 15, 2023
* HTMLAudioElementを使わないようにする

* fix CHANGELOG.md

---------

Co-authored-by: osamu <46447427+sam-osamu@users.noreply.github.com>
@jojobii-arks
Copy link
Contributor

oh hell yeah. thank you for this 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants