Skip to content

fix(iTip): Prevent single-occurrence reply to be a significantChange for everyone - #784

Open
DerDreschner wants to merge 1 commit into
sabre-io:masterfrom
DerDreschner:fix/prevent-mail-fan-out
Open

DerDreschner wants to merge 1 commit into
sabre-io:masterfrom
DerDreschner:fix/prevent-mail-fan-out

Conversation

@DerDreschner

Copy link
Copy Markdown

Under some circumstances (e.g., replying to a single event in a recurring series with a lot of participants), all participants get a new REQUEST message delivered. It's pretty confusing for them, as this shouldn't happen. This is being fixed with this PR.

@DerDreschner

Copy link
Copy Markdown
Author

@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.11765% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.61%. Comparing base (fb5863d) to head (cd9878c).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
lib/ITip/Broker.php 94.11% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #784      +/-   ##
============================================
- Coverage     98.66%   98.61%   -0.05%     
- Complexity     1759     1783      +24     
============================================
  Files            71       71              
  Lines          5165     5213      +48     
============================================
+ Hits           5096     5141      +45     
- Misses           69       72       +3     

☔ View full report in Codecov by Harness.
📢 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.

…` for everyone

Assisted-by: ClaudeCode:claude-fable-5
Signed-off-by: David Dreschner <david.dreschner@nextcloud.com>
@DerDreschner
DerDreschner force-pushed the fix/prevent-mail-fan-out branch from cd9878c to 303afac Compare August 17, 2026 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant