Skip to content

docs: remove Microsoft Teams (msteamsbridge) integration docs#1561

Merged
phil-davis merged 2 commits into
masterfrom
chore/remove-msteamsbridge-docs
Jul 6, 2026
Merged

docs: remove Microsoft Teams (msteamsbridge) integration docs#1561
phil-davis merged 2 commits into
masterfrom
chore/remove-msteamsbridge-docs

Conversation

@DeepDiver1975

@DeepDiver1975 DeepDiver1975 commented Jul 6, 2026

Copy link
Copy Markdown
Member

owncloud/msteamsbridge is being archived and is no longer maintained (heads-up: owncloud/docs#5125). This removes its documentation from the server docs.

Changes

  • Delete the two MS-Teams integration pages:
    • modules/classic_ui/pages/integration/ms-teams.adoc (end-user)
    • modules/admin_manual/pages/configuration/integration/ms-teams.adoc (admin)
  • Delete the now-empty classic_ui Integration section (integration/index.adoc, which only listed the Teams page) and remove the Integration block from classic_ui nav.
  • admin_manual: drop the Teams entry from the Integration index and nav — the Splunk integration remains, so the section stays.
  • Remove the == App: Microsoft Teams Bridge config sample from config_apps_sample_php_parameters.adoc.
  • Reword the .htaccess note in manual_upgrade.adoc that xref'd the removed Teams page (kept the WebDav example).

Verification

  • grep for msteamsbridge|msteamsgen|ms-teams|microsoft teams across all .adoc files → 0 matches after the change.
  • No remaining xref points at any deleted page (the surviving configuration/integration/index.adoc xref targets the admin index, which is retained for Splunk).

Note: the heads-up issue was filed against owncloud/docs, but that repo is only the Antora aggregator — the actual content lives here in owncloud/docs-server.

🤖 Generated with Claude Code


Also resolves owncloud/docs#5126 (msteamsgen heads-up): the MS-Teams generator and Bridge App are two halves of the same integration and share these same pages, so this removal covers both. Verified grep includes msteamsgen → 0 matches.

owncloud/msteamsbridge is being archived and is no longer maintained
(see owncloud/docs#5125). Remove its documentation:

- Delete the end-user and admin MS-Teams integration pages.
- Delete the now-empty classic_ui Integration section (it only held the
  Teams page) and drop it from the classic_ui nav.
- Drop the Teams entry from the admin_manual Integration index and nav
  (the Splunk integration remains).
- Remove the 'App: Microsoft Teams Bridge' config sample from the app
  parameters reference.
- Reword the manual-upgrade .htaccess note that xref'd the removed page.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
@mmattel

mmattel commented Jul 6, 2026

Copy link
Copy Markdown
Contributor
  • Please remove the relevant data of config_apps_sample_php_parameters in core. I will check that this is then properly removed here in this PR after the core PR is merged. Otherwise any config-to-docs run will re-add the content...

  • There are images that need to be removed too. I will take care.

  • I also will take care to remove it from the app list

@DeepDiver1975 DeepDiver1975 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve (review-only — self-authored). Clean removal; Antora build safe.

Dangling-xref check (grepped the full PR-head tree 455ae39):

  • Zero remaining xref:/link:/image: references to either deleted ms-teams.adoc, or to the deleted classic_ui/integration/index.adoc, or to msteamsbridge/msteamsgen/"Microsoft Teams".
  • admin_manual/partials/nav.adoc: only the ms-teams child dropped; integration/index + splunk retained — correct.
  • classic_ui/partials/nav.adoc: whole integration section removed (index+ms-teams) — correct, MS Teams was its only content.
  • manual_upgrade.adoc:60 and config_apps_sample_php_parameters.adoc (msteamsbridge section): edits are MS-Teams-scoped, surrounding content intact — coherent.

Non-blocking: orphaned image assets remain and are now unreferenced — admin_manual/images/configuration/integration/ms-teams/ (10 files) and classic_ui/images/integration/ms-teams/ (2). Doesn't break the build; delete in a follow-up (or here) to avoid dead assets.

CI green (Build documentation, lint).

🤖 Generated with Claude Code

@DeepDiver1975

Copy link
Copy Markdown
Member Author

Good catch on the config sample — you're right that config_apps_sample_php_parameters.adoc is generated (everything below // header end do not delete or edit this line), so a config-to-docs run would re-add the Teams Bridge block from core.

I've opened the root-cause fix in core: owncloud/core#41668 — it removes the msteamsbridge block from config/config.apps.sample.php (plus a changelog entry).

Once that merges and a config-to-docs sync runs, the generated .adoc here will drop the block cleanly. Happy to revert the manual edit to the generated file in this PR if you'd prefer to let the sync handle it — let me know which you'd like.

The images and app-list removal are yours as noted. 🙏

DeepDiver1975 added a commit to owncloud/core that referenced this pull request Jul 6, 2026
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>
The MS-Teams integration pages were deleted in this PR; remove their now
orphaned image assets (admin_manual and classic_ui) as well.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
@DeepDiver1975

Copy link
Copy Markdown
Member Author

Took a couple of these off your plate since we were already in here:

  • Images removed — pushed a commit deleting the 12 orphaned MS-Teams image assets (10 under admin_manual/images/configuration/integration/ms-teams/, 2 under classic_ui/images/integration/ms-teams/). All were fully de-referenced once the pages were deleted; grep for ms-teams/ in .adoc → 0 matches.
  • Config sample — root cause is in core, handled separately in chore: remove msteamsbridge config sample (app archived) core#41668 (removes the msteamsbridge block from config.apps.sample.php so config-to-docs won't re-add it).

Left for you, since it's outside this repo:

  • App list / marketplace — I couldn't find an in-repo "app list" to edit (no .adoc enumerates it), so this looks like the external marketplace.owncloud.com listing, which needs your access. Flagging rather than guessing — let me know if you actually meant an in-repo file and I'll take it.

phil-davis pushed a commit to owncloud/core that referenced this pull request Jul 6, 2026
* chore: remove msteamsbridge config sample (app archived)

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>

* chore: update changelog

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>

---------

Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
@phil-davis phil-davis merged commit 7b9dfd8 into master Jul 6, 2026
2 checks passed
@phil-davis phil-davis deleted the chore/remove-msteamsbridge-docs branch July 6, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

msteamsgen: going out of maintenance

3 participants