Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

🦋 Changeset detected

Latest commit: 8a5703e

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Walkthrough

This PR removes the deprecated livechat:saveUnit Meteor server method from the livechat-enterprise module. The removal includes deleting the method implementation file, eliminating the corresponding import, and adding a changeset documenting the removal as part of a major release.

Changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Straightforward removal of deprecated code with no replacement logic
• Changes are consistent deletions across related files
• No conditional logic or complex refactoring to evaluate

Possibly related PRs

• chore: Adds deprecation warning for livechat:saveUnit #37255: Adds deprecation warning and client migration paths for livechat:saveUnit before its removal

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

• ggazzo
• dougfabris
• MarteinSchoeler

Poem

🐰 A method deprecated, now laid to rest,
The cleanup removes what once was blessed,
No longer shall saveUnit call,
A cleaner codebase, best of all! 🌙

Pre-merge checks and finishing touches

Learnt from: rodrigok
Repo: RocketChat/Rocket.Chat PR: 36991
File: apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts:219-221
Timestamp: 2025-09-19T15:15:04.642Z
Learning: The Federation_Matrix_homeserver_domain setting in apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts is part of the old federation system and is being deprecated/removed, so configuration issues with this setting should not be flagged for improvement.

Learnt from: rodrigok
Repo: RocketChat/Rocket.Chat PR: 36991
File: apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts:219-221
Timestamp: 2025-09-19T15:15:04.642Z
Learning: The Federation_Matrix_homeserver_domain setting in apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts is part of the old federation system and is being deprecated/removed, so configuration issues with this setting should not be flagged for improvement.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.92%. Comparing base (750af59) to head (8a5703e).
⚠️ Report is 1 commits behind head on release-8.0.0.

@@                Coverage Diff                @@
##           release-8.0.0   #37461      +/-   ##
=================================================
+ Coverage          68.44%   70.92%   +2.47%     
=================================================
  Files               2877     3035     +158     
  Lines             101867   104621    +2754     
  Branches           17812    18435     +623     
=================================================
+ Hits               69727    74198    +4471     
+ Misses             30493    28478    -2015     
- Partials            1647     1945     +298