chore: remove msteamsbridge config sample (app archived)#41668
Merged
Conversation
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
The owncloud/msteamsbridge integration app is being archived and is no longer maintained. Remove its Microsoft Teams Bridge config sample block from config.apps.sample.php so the config-to-docs sync no longer re-adds the parameters to the admin documentation. Related: owncloud/docs#5125, owncloud/docs-server#1561 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
7848437 to
b9eb1fd
Compare
phil-davis
approved these changes
Jul 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
owncloud/msteamsbridgeintegration app is being archived and is no longer maintained (heads-up: owncloud/docs#5125).This removes the Microsoft Teams Bridge (
msteamsbridge) config sample block fromconfig/config.apps.sample.php.Why this belongs in core
config.apps.sample.phpis the source that the config-to-docs tool converts into the admin documentation pageconfig_apps_sample_php_parameters.adoc. The docs cleanup in owncloud/docs-server#1561 removed the rendered block from that generated.adoc, but everything below its// header end do not delete or edit this linemarker is generated — so without removing the source here, the next config-to-docs sync would re-add the Teams Bridge parameters.Changes
App: Microsoft Teams Bridgecomment block and the'msteamsbridge' => ['loginButtonName' => ...]array fromconfig/config.apps.sample.php.Verification
php -l config/config.apps.sample.php→ no syntax errors.grep -i msteams config/config.apps.sample.php→ 0 matches after the change.Related: owncloud/docs#5125, owncloud/docs-server#1561
🤖 Generated with Claude Code