Skip to content

AI-Aided: Remove built-in grace period references from documentation #8066

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

Merged
merged 4 commits into from
Jun 19, 2025

Conversation

lieut-data
Copy link
Member

@lieut-data lieut-data commented Jun 17, 2025

Summary

  • Removed references to the built-in 5% grace period from the licensed user limit error documentation
  • Updated error conditions to reflect current behavior without grace period

Related PRs

Test plan

  • Verify documentation accurately reflects current licensing behavior
  • Review that all grace period references have been removed

🤖 Generated with Claude Code

The fixed 5% grace period for user seat limits has been replaced with a
configurable ExtraUsers field in PR #31629. This updates the documentation
to remove references to the built-in grace period behavior.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@lieut-data lieut-data force-pushed the mm-31628-remove-builtin-grace-docs branch from 0b60677 to bb30e29 Compare June 17, 2025 16:06
Copy link

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

@lieut-data lieut-data marked this pull request as ready for review June 17, 2025 16:06
Copy link

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

Copy link

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

Copy link

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

@cwarnermm cwarnermm added the 1: Dev Review Requires review by a core commiter label Jun 17, 2025
@cwarnermm cwarnermm changed the title Remove built-in grace period references from documentation AI-Aided: Remove built-in grace period references from documentation Jun 18, 2025
@cwarnermm cwarnermm added this to the v10.10.0 milestone Jun 18, 2025
Copy link

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

@cwarnermm cwarnermm added 3: Reviews Complete All reviewers have approved the pull request and removed 1: Dev Review Requires review by a core commiter labels Jun 19, 2025
@cwarnermm
Copy link
Member

@lieut-data - Any concerns about me moving this to the `v10.10-documentation`` branch for publication with v10.10?

@lieut-data
Copy link
Member Author

@cwarnermm, no objections -- is this actually a pattern we should adopt going forward? I admit I was ignorant about branch specific updates until now. (Or possibly I'd just forgotten :))

Note too that I /think/ the previous version of this docs is already live, so not sure if we want to revert that or not.

@cwarnermm
Copy link
Member

@lieut-data - Documenting product functionality within the associated release branch helps Docs and Release Management manage the monthly maintenance updates and is the preferred approach.

In this specific case, given that the content is already published/live, I recommend publishing now rather than reverting and publishing in July.

@lieut-data lieut-data merged commit 95898fc into master Jun 19, 2025
5 checks passed
@lieut-data lieut-data deleted the mm-31628-remove-builtin-grace-docs branch June 19, 2025 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants