Skip to content

Conversation

@uni-kakurenbo
Copy link
Contributor

概要

  • とりあえず delay 10 秒の debouncing に加えて以下の場合にリクエストを飛ばします
    • ページ遷移があったとき (beforeunload イベントが発火したとき)
    • チャンネルの遷移があったとき (現在のビューの channelId が変化したとき)

なぜこの PR を入れたいのか

closes: #4728

PR を出す前の確認事項

  • (機能の追加なら)追加することの合意がチームで取れている
    • 取れていない場合はチェックを外して PR にすれば OK
  • 動作確認ができている
  • 自分で一度コードを眺めて自分的に問題はなさそう

@github-actions
Copy link

@uni-kakurenbo uni-kakurenbo changed the title fix: 一定時間内の通知設定の再変更は、チャンネルのイベントログに記録を残さないようにする fix: 短時間での通知設定の再変更がイベントログに記録を残さないようにする Sep 25, 2025
@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

❌ Patch coverage is 2.00000% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 13.49%. Comparing base (90cf7b9) to head (39811b9).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
src/composables/utils/useDebounceWithAutoFlush.ts 0.00% 40 Missing ⚠️
...sables/subscription/useChannelSubscriptionState.ts 0.00% 9 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4826   +/-   ##
=======================================
  Coverage   13.49%   13.49%           
=======================================
  Files         703      704    +1     
  Lines       31527    31590   +63     
  Branches      673      676    +3     
=======================================
+ Hits         4253     4262    +9     
- Misses      27263    27316   +53     
- Partials       11       12    +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@uni-kakurenbo uni-kakurenbo force-pushed the fix/suppress-change-notification-event-log branch from 3f90709 to c07e15e Compare September 25, 2025 05:00
@uni-kakurenbo uni-kakurenbo self-assigned this Sep 25, 2025
@uni-kakurenbo uni-kakurenbo requested a review from a team September 25, 2025 05:01
@uni-kakurenbo uni-kakurenbo changed the title fix: 短時間での通知設定の再変更がイベントログに記録を残さないようにする fix: 短時間での通知設定の再変更がイベントログに複数の記録を残さないようにする Sep 25, 2025
@uni-kakurenbo uni-kakurenbo force-pushed the fix/suppress-change-notification-event-log branch from c07e15e to dd2cb35 Compare September 25, 2025 05:08
@uni-kakurenbo uni-kakurenbo changed the title fix: 短時間での通知設定の再変更がイベントログに複数の記録を残さないようにする fix(channel-view): 短時間での通知設定の再変更がイベントログに複数の記録を残さないようにする Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

通知設定をクリック2回で未読→通知なしに切り替えるときに、イベントログを残したくない

2 participants