Skip to content

Update china.md #4807

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update china.md #4807

wants to merge 3 commits into from

Conversation

sandytsang
Copy link
Contributor

Update correct method how to install Company Portal app. Added Feature update and Driver update is not available in 21Vianet tenant.

Update correct method how to install Company Portal app. Added Feature update and Driver update is not available in 21Vianet tenant.
Copy link
Contributor

@sandytsang : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 5cfbfe7:

⚠️ Validation status: warnings

File Status Preview URL Details
intune/intune-service/fundamentals/china.md ⚠️Warning Details

intune/intune-service/fundamentals/china.md

  • Line 53, Column 102: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/intune/intune-service/apps/store-apps-company-portal-app#download-the-offline-company-portal-app' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 53, Column 102: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/intune/intune-service/apps/store-apps-company-portal-app#download-the-offline-company-portal-app' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

lenewsad added 2 commits July 23, 2025 13:36
Style updates
Updated link line 53
Copy link
Contributor

Learn Build status updates of commit c39e64e:

⚠️ Validation status: warnings

File Status Preview URL Details
intune/intune-service/fundamentals/china.md ⚠️Warning Details

intune/intune-service/fundamentals/china.md

  • Line 53, Column 102: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/intune/intune-service/apps/store-apps-company-portal-app#download-the-offline-company-portal-app' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 53, Column 102: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/intune/intune-service/apps/store-apps-company-portal-app#download-the-offline-company-portal-app' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

@lenewsad lenewsad left a comment

Choose a reason for hiding this comment

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

Thank you @sandytsang for your contribution to the Microsoft Intune docs!

Copy link
Contributor

Learn Build status updates of commit c0664b8:

✅ Validation status: passed

File Status Preview URL Details
intune/intune-service/fundamentals/china.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.

@lenewsad lenewsad self-assigned this Jul 23, 2025
@lenewsad lenewsad added the pm Request PM help to resolve the issue label Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants