Skip to content

DocsBrokenWindow: Add missing DataRetentionSettings configuration settings #8169

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

DocsBrokenWindow: Add missing DataRetentionSettings configuration settings #8169

wants to merge 4 commits into from
+126 −0

Conversation

@cwarnermm
Copy link
Member

@cwarnermm cwarnermm commented Jul 7, 2025

Add 7 missing DataRetentionSettings configuration options to the compliance documentation:

  • TimeBetweenBatchesMilliseconds
  • RetentionIdsBatchSize
  • BoardsRetentionDays
  • EnableMessageDeletion
  • EnableFileDeletion
  • EnableBoardsDeletion
  • BatchSize

These settings provide better control over data retention job execution and performance.

Fixes #8160

Generated with Claude Code

@claude @cwarnermm
Add missing DataRetentionSettings configuration settings
18615ca 
Add 7 missing DataRetentionSettings configuration options to the compliance documentation:
- TimeBetweenBatchesMilliseconds
- RetentionIdsBatchSize  
- BoardsRetentionDays
- EnableMessageDeletion
- EnableFileDeletion
- EnableBoardsDeletion
- BatchSize

These settings provide better control over data retention job execution and performance.

Fixes #8160

Co-authored-by: Carrie Warner (Mattermost) <cwarnermm@users.noreply.github.com>
@cwarnermm cwarnermm added the 2: Editor Review Requires review by an editor label Jul 7, 2025
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

Newest code from mattermost has been published to preview environment for Git SHA 18615ca

@cwarnermm
Copy link
Member Author

cwarnermm commented Jul 8, 2025

Next steps:

  • Incorporate default & possible values
  • Ensure caveats are formatted as notes below each table
  • Review search metadata

@github-actions
Copy link

github-actions bot commented Jul 10, 2025

Newest code from mattermost has been published to preview environment for Git SHA 5b704d1

@cwarnermm
Copy link
Member Author

cwarnermm commented Jul 10, 2025

TimeBetweenBatchesMilliseconds

  • Default: 100 milliseconds
  • Possible Values: Any positive integer (≥ 0)

RetentionIdsBatchSize

  • Default: 100
  • Possible Values: Any positive integer (≥ 0)

BoardsRetentionDays

  • Default: 365 days
  • Possible Values: Any non-negative integer (≥ 0)

EnableMessageDeletion

  • Default: false
  • Possible Values: true or false

EnableFileDeletion

  • Default: false
  • Possible Values: true or false

EnableBoardsDeletion

  • Default: false
  • Possible Values: true or false

BatchSize

  • Default: 3000
  • Possible Values: Any positive integer (≥ 0)

@cwarnermm
Copy link
Member Author

cwarnermm commented Jul 10, 2025

@claude - Incorporate the default and possible configuration setting values specified via #8169 (comment)

@claude
Copy link
Contributor

claude bot commented Jul 10, 2025 •
edited
Loading

Claude finished @cwarnermm's task —— View job

Todo List:

  • Read current documentation file to understand structure
  • Incorporate default and possible values for the 7 settings:
    • TimeBetweenBatchesMilliseconds (added possible values info)
    • RetentionIdsBatchSize (updated default from 1000 to 100)
    • BoardsRetentionDays (updated default from 0 to 365 days)
    • EnableMessageDeletion (updated default from true to false)
    • EnableFileDeletion (updated default from true to false)
    • EnableBoardsDeletion (updated default from true to false)
    • BatchSize (updated default from 1000 to 3000)
  • Commit and push changes
  • Update comment with completion status

Summary

Successfully updated all 7 DataRetentionSettings configuration options in source/configure/compliance-configuration-settings.rst with the provided default values and possible values information:

Updated default values:

  • RetentionIdsBatchSize: 1000 → 100
  • BoardsRetentionDays: 0 → 365 days
  • EnableMessageDeletion: true → false
  • EnableFileDeletion: true → false
  • EnableBoardsDeletion: true → false
  • BatchSize: 1000 → 3000

Added possible values information:

  • All settings now include their possible value ranges in the documentation

The changes have been committed and pushed to the PR branch: claude/issue-8160-20250707_202547

@claude @cwarnermm
Update DataRetentionSettings configuration defaults and add possible …
67e6daa 
…values

- Updated RetentionIdsBatchSize default from 1000 to 100
- Updated BoardsRetentionDays default from 0 to 365 days
- Updated EnableMessageDeletion default from true to false
- Updated EnableFileDeletion default from true to false
- Updated EnableBoardsDeletion default from true to false
- Updated BatchSize default from 1000 to 3000
- Added possible values information for all settings

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

github-actions bot commented Jul 10, 2025

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

@github-actions
Copy link

github-actions bot commented Jul 10, 2025

Newest code from mattermost has been published to preview environment for Git SHA 67e6daa

@cwarnermm cwarnermm changed the title Add missing DataRetentionSettings configuration settings DocsBrokenWindow: Add missing DataRetentionSettings configuration settings Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

2: Editor Review Requires review by an editor Guidance

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Docs Broken Window: Add missing DataRetentionSettings configuration settings

2 participants

@cwarnermm