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(frontend): クライアント設定から13.7.0で削除されたチャット機能に関するサウンド設定を削除 #11912

Merged

Conversation

camilla-ett
Copy link
Contributor

@camilla-ett camilla-ett commented Sep 28, 2023

What

クライアント設定->サウンド から以下の2項目を削除した

  • チャット
  • チャット(バックグラウンド)

before
before

after
after

Why

チャット機能は 13.7.0 で削除されており (https://misskey-hub.net/docs/releases.html#_13-7-0
サウンド設定だけがクライアント設定に残っていたため、削除した。

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 Sep 28, 2023
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (5edc885) 78.93% compared to head (2bdeb81) 78.67%.
Report is 22 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11912      +/-   ##
===========================================
- Coverage    78.93%   78.67%   -0.27%     
===========================================
  Files          928      176     -752     
  Lines        99069    23144   -75925     
  Branches      7888      511    -7377     
===========================================
- Hits         78202    18208   -59994     
+ Misses       20867     4936   -15931     
Files Coverage Δ
packages/frontend/src/scripts/sound.ts 81.48% <ø> (-1.04%) ⬇️

... and 756 files with indirect coverage changes

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

@camilla-ett camilla-ett force-pushed the remove_clinet_setting_sound_chat branch from 9cbe93d to fb55d88 Compare September 28, 2023 08:18
@camilla-ett camilla-ett changed the title fix(frontend): クライアント設定->サウンド->チャット等を削除 fix(frontend): クライアント設定から13.7.0で削除されたチャット機能に関するサウンド設定を削除 Sep 28, 2023
@camilla-ett camilla-ett marked this pull request as ready for review September 28, 2023 08:36
@camilla-ett camilla-ett force-pushed the remove_clinet_setting_sound_chat branch from b162b35 to ada99ec Compare September 28, 2023 10:58
@syuilo
Copy link
Member

syuilo commented Sep 29, 2023

チャット機能が復活する可能性もないことはないからUI上から設定項目を消すだけ(ローカリゼーションや設定自体は残す)で良さそうと思ったけどどうだろ

@syuilo
Copy link
Member

syuilo commented Sep 29, 2023

チャット機能が復活する可能性

DMノートをチャット風UIで表示する機能が実装される可能性は大いにある

@tai-cha
Copy link
Contributor

tai-cha commented Sep 29, 2023

復活させるときにRevertすればいい説もあるといえばありそう

@camilla-ett
Copy link
Contributor Author

camilla-ett commented Sep 30, 2023

DMノートをチャット風UIで表示する機能が実装される可能性は大いにある

だとすれば、おそらくそれはチャットでは無く、今ある機能の名前で言うとダイレクトノートなので、chat chatBg ではない気がしますがどうでしょう…?

@camilla-ett
Copy link
Contributor Author

locales を残す場合は、 45f15bd のコミットだけchery-pickすれば良いので、どちらでも……!

@camilla-ett
Copy link
Contributor Author

#11962
locales をいじっていない版

@syuilo syuilo merged commit e4ade46 into misskey-dev:develop Oct 4, 2023
14 of 15 checks passed
@syuilo
Copy link
Member

syuilo commented Oct 4, 2023

🙏

@camilla-ett camilla-ett deleted the remove_clinet_setting_sound_chat branch October 4, 2023 20:19
kanarikanaru pushed a commit to kanarikanaru/misskey that referenced this pull request Oct 12, 2023
…#11912)

* fix(frontend): クライアント設定から13.7.0で削除されたチャット機能に関するサウンド設定を削除

* fix(frontend): 各localesからsfx/chat sfx/chatbgを削除
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.

4 participants