Skip to content

docs: show Intune MAM on Entry and Enterprise Advanced only#8986

Open
marianunez wants to merge 1 commit into
masterfrom
cursor/intune-mam-plan-availability-ba0e
Open

docs: show Intune MAM on Entry and Enterprise Advanced only#8986
marianunez wants to merge 1 commit into
masterfrom
cursor/intune-mam-plan-availability-ba0e

Conversation

@marianunez
Copy link
Copy Markdown
Member

@marianunez marianunez commented May 18, 2026

Updates Intune MAM plan availability documentation to reflect that the feature is available on Entry and Enterprise Advanced plans, not the Enterprise plan.

Changes

Open in Web Open in Cursor 

Swap the Intune MAM configuration guide plan badge from entry-ent to
entry-adv, and remove the Enterprise column checkmark from the Intune
MAM row in the plans overview table.

Co-authored-by: Maria A Nunez <maria.nunez@mattermost.com>
@marianunez marianunez marked this pull request as ready for review May 18, 2026 21:46
@marianunez marianunez requested review from Copilot and enahum May 18, 2026 21:46
@github-actions
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Mattermost documentation to show Microsoft Intune MAM availability as Entry and Enterprise Advanced only, removing Enterprise plan availability.

Changes:

  • Updates the Intune MAM guide badge from Entry/Enterprise/Enterprise Advanced to Entry/Enterprise Advanced.
  • Removes the Enterprise plan checkmark from the plans overview table for Microsoft Intune MAM Support.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
source/deployment-guide/mobile/configure-microsoft-intune-mam.rst Updates the page availability badge for Intune MAM.
source/product-overview/plans.md Updates the plans comparison row for Intune MAM availability.

==============================================================

.. include:: ../../_static/badges/entry-ent.rst
.. include:: ../../_static/badges/entry-adv.rst
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.

No changes are required considering Entry is not a license it is uploaded and is the default plan.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This PR updates documentation to reflect changes in Microsoft Intune MAM Support feature availability. The feature availability matrix is updated to remove Enterprise edition support while retaining Entry and Enterprise Advanced. The corresponding deployment guide badge reference is updated to match the revised availability scope.

Changes

Microsoft Intune MAM Feature Availability

Layer / File(s) Summary
Feature availability matrix update
source/product-overview/plans.md
The "Microsoft Intune MAM Support" row removes the check mark from the Enterprise column while maintaining availability indicators for Entry and Enterprise Advanced editions.
Deployment guide badge reference
source/deployment-guide/mobile/configure-microsoft-intune-mam.rst
The badge include is updated from entry-ent.rst to entry-adv.rst to align with the revised feature availability scope.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: updating documentation to reflect that Intune MAM is available only on Entry and Enterprise Advanced plans.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The pull request description clearly documents the purpose and changes, directly relating to the changeset of updating Microsoft Intune MAM plan availability documentation.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/intune-mam-plan-availability-ba0e

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@source/product-overview/plans.md`:
- Line 271: The two <img> elements using
"../_static/images/check-circle-green.svg" lack alt attributes; update each <img
src="../_static/images/check-circle-green.svg"> to include a descriptive alt
such as alt="Microsoft Intune MAM Support included" (or a concise phrase like
"Included") so screen readers can convey that the plan includes Intune MAM
Support; ensure both occurrences are updated to use the same descriptive alt
text for consistency.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 83b6675a-887c-4e49-8ea2-0c871350f7cf

📥 Commits

Reviewing files that changed from the base of the PR and between cf048c8 and f7435c0.

📒 Files selected for processing (2)
  • source/deployment-guide/mobile/configure-microsoft-intune-mam.rst
  • source/product-overview/plans.md

Comment thread source/product-overview/plans.md
@Combs7th Combs7th added the 2: Editor Review Requires review by an editor label May 20, 2026
@Combs7th Combs7th requested review from Combs7th and esethna May 20, 2026 02:13
@Combs7th
Copy link
Copy Markdown
Contributor

LGTM. @esethna - Ready for merge?

@marianunez
Copy link
Copy Markdown
Member Author

LGTM. @esethna - Ready for merge?

There is still some discussions on this here https://hub.mattermost.com/private-core/pl/d3s1t5np8pgk3fxcaizaf6urbr

Do not merge yet

@esethna esethna added the Do Not Merge Should not be merged until this label is removed label May 20, 2026
@marianunez marianunez removed the Do Not Merge Should not be merged until this label is removed label May 20, 2026
@marianunez
Copy link
Copy Markdown
Member Author

@esethna we will move forward with this update. We can merge if it looks good to you

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.

6 participants