fix: Missing retention's ignoreThreads param for old channels

[dougfabris]

Proposed changes (including videos or screenshots)

Introduced here: #32485

While introducing this new possibility in retention policy overriding feature, old channels were not being affected, because it was missing the ignoreThreads parameter. So we decide to add a migration to add the missing parameter in all channels with retention policy override.

Issue(s)

Closes #32794

Steps to test or reproduce

Further comments

CORE-613

[dionisio-bot]

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

[changeset-bot]

🦋 Changeset detected

Latest commit: 3d82e07

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

This PR includes changesets to release 32 packages

Name	Type
@rocket.chat/meteor	Patch
@rocket.chat/core-typings	Patch
@rocket.chat/rest-typings	Patch
@rocket.chat/apps	Patch
@rocket.chat/core-services	Patch
@rocket.chat/cron	Patch
@rocket.chat/fuselage-ui-kit	Patch
@rocket.chat/gazzodown	Patch
@rocket.chat/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/ui-contexts	Patch
@rocket.chat/account-service	Patch
@rocket.chat/authorization-service	Patch
@rocket.chat/ddp-streamer	Patch
@rocket.chat/omnichannel-transcript	Patch
@rocket.chat/presence-service	Patch
@rocket.chat/queue-worker	Patch
@rocket.chat/stream-hub-service	Patch
@rocket.chat/api-client	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/license	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/presence	Patch
rocketchat-services	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/models	Patch
@rocket.chat/ui-avatar	Patch
@rocket.chat/ui-client	Patch
@rocket.chat/ui-video-conf	Patch
@rocket.chat/web-ui-registration	Patch
@rocket.chat/instance-status	Patch

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.39%. Comparing base (8c447cf) to head (3d82e07).
Report is 7 commits behind head on release-7.0.0.

Additional details and impacted files

@@              Coverage Diff               @@
##           release-7.0.0   #33028   +/-   ##
==============================================
  Coverage          59.39%   59.39%           
==============================================
  Files               2541     2541           
  Lines              63175    63175           
  Branches           14220    14220           
==============================================
  Hits               37525    37525           
  Misses             22935    22935           
  Partials            2715     2715           

Flag	Coverage Δ
unit	76.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[gabriellsh]

As we discussed previously, we shouldn't use the setting value as the default value since it would affect the current behaviour. Even though the current behaviour is "wrong", we would be breaking user's expectations and this could be harmful.

We should proceed just with the migration for the next major version.