Skip to content

AI-Aided: Add data retention support for preserving pinned posts #8071

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: v10.10-documentation
Choose a base branch
from

Conversation

cwarnermm
Copy link
Member

@cwarnermm cwarnermm commented Jun 18, 2025

Add documentation for the new data retention feature that optionally preserves pinned posts, available from Mattermost server v10.10 onward via mattermost/mattermost#31165

Changes

  • Add Preserve pinned posts configuration setting in compliance settings
  • Update data retention policy documentation to reflect pinned post preservation option
  • Document config.json setting and environment variable

Closes #8070

Generated with Claude Code

amyblais and others added 4 commits June 18, 2025 12:13
* Update mattermost-mobile-releases.md

* Update mobile-app-changelog.md

* Update mobile-app-changelog.md

---------

Co-authored-by: Carrie Warner (Mattermost) <74422101+cwarnermm@users.noreply.github.com>
Co-authored-by: Carrie Warner (Mattermost) <74422101+cwarnermm@users.noreply.github.com>
- Add Preserve pinned posts configuration setting in compliance settings
- Update data retention policy documentation to reflect pinned post preservation option
- Available from Mattermost server v10.10 onward

Closes #8070

Co-authored-by: Carrie Warner (Mattermost) <cwarnermm@users.noreply.github.com>
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 082ca2b

@cwarnermm
Copy link
Member Author

@claude - Concisely explain why this PR contains changes to other parts of the repository beyond what was asked.

Copy link
Contributor

claude bot commented Jun 18, 2025

Claude finished @cwarnermm's task —— View job


Analysis of PR Changes

Investigating why this PR contains changes beyond the data retention feature scope.

Todo List:

  • Read data retention related files to understand core changes
  • Read mobile app changelog/release files to understand those changes
  • Read subscription and deployment files to understand those changes
  • Categorize changes and identify what's unrelated to the original task
  • Provide concise explanation

Analysis Complete

The PR contains changes to 6 files, but only 2 are directly related to the data retention feature for preserving pinned posts:

Related to Issue #8070 (Data Retention):

  • source/comply/data-retention-policy.rst - Added documentation for the preserve pinned posts feature
  • source/configure/compliance-configuration-settings.rst - Added configuration setting documentation

Unrelated changes that were included inadvertently:

  • source/about/mattermost-mobile-releases.md - Mobile app release table updates
  • source/about/mobile-app-changelog.md - Mobile app v2.29.1 changelog entries
  • source/about/subscription.rst - Subscription documentation updates
  • source/deploy/server/deploy-linux.rst - Linux deployment documentation changes

Explanation

This PR contains changes to other parts of the repository because the branch was created from a point that included recent commits for mobile releases (#8060), subscription updates (#8069), and Linux deployment reorganization (#8025). When the data retention feature work was added on top of these commits, all the changes became part of this PR.

The branch should have been created from a clean main/master branch state containing only the data retention changes, or the unrelated commits should have been excluded from the PR scope.


@cwarnermm cwarnermm added this to the v10.10.0 milestone Jun 18, 2025
@cwarnermm cwarnermm added the 2: Editor Review Requires review by an editor label Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Editor Review Requires review by an editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants