sw: なんかもうめっちゃ変えた

[tamaina]

What

• 「通知が既読になったらプッシュ通知を削除する」の機能だけが殺されたのを復活させる（設定だけ残っていたが、設定を完全に消すのが面倒だったため機能を復活させることにした）
  • user_visible_auto_notification防止用の通知の挙動を変えた（hostをタイトルに・bodyにバージョンを埋め込み、削除はしないように）
• プッシュ通知が実績を解除 (achievementEarned) に対応
• notificationclickで既存のクライアントの投稿フォームを開く(order: post)際の挙動を修正
• たくさんのプッシュ通知を閉じた際、その通知の数だけnotifications/mark-all-as-readを叩くのをやめるように

Why

プッシュ通知をいい感じにしたい

Checklist

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

[codecov]

Codecov Report

Merging #10570 (db4ec72) into develop (da64273) will increase coverage by 0.04%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           develop   #10570      +/-   ##
===========================================
+ Coverage    75.10%   75.14%   +0.04%     
===========================================
  Files          886      888       +2     
  Lines        87116    87281     +165     
  Branches      5906     5902       -4     
===========================================
+ Hits         65430    65591     +161     
- Misses       21686    21690       +4     

Impacted Files	Coverage Δ
...ckages/backend/src/core/PushNotificationService.ts	38.31% <20.00%> (-0.90%)	⬇️
packages/backend/src/core/NotificationService.ts	93.67% <100.00%> (+0.04%)	⬆️

... and 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[tamaina]

user_visible_auto_notification防止用の通知

iconにインスタンスのアイコンを埋め込んでもよさそうな気もするが、iOSだと二重に表示されそうだし表示されるたびに/metaなどを取りに行きまくる感じになるので微妙そう

[syuilo]

🙏🏻