Skip to content
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

Fix Outdated HTTP with Entra ID Details #2439

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

Conversation

ryanperrymba
Copy link

@ryanperrymba ryanperrymba commented Feb 6, 2024

These instructions for configuring HTTP with Entra ID connection were out of date. The newer connector is not preauthorized for Graph API, and so throws errors if you attempt to use it without granting it admin consent.

Updated Sample Audit Process document.  Initial use of term 'Audit' was vague, as was not clear what auditing was implied, and conflicted with other true Auditing functionality. This article is really more about enforcing compliance, which includes, but is not limited to auditing.
Copy link

Learn Build status updates of commit e0d6291:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit e0d6291f234cdb6a55816110037ccce3e0c0b84a in branch fixhttpazureaddetails of repository https://github.com/ryanperrymba/MicrosoftDocs-power-platform into branch main (commit 657eabba93a7a48b61090f134ba724067cd2d5d0). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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.

For any questions, please:

@ryanperrymba
Copy link
Author

Note: Includes other changes previously submitted on PR #2288.

Copy link

Learn Build status updates of commit 731ab6c:

⚠️ Validation status: warnings

File Status Preview URL Details
power-platform/guidance/coe/setup-core-components.md ⚠️Warning Details
power-platform/guidance/coe/example-processes.md ✅Succeeded
power-platform/guidance/coe/media/httpazuread.png ✅Succeeded

power-platform/guidance/coe/setup-core-components.md

  • Line 61, Column 7: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/connectors/webcontents/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 61, Column 239: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/connectors/webcontentsv2/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 61, Column 7: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/connectors/webcontents/' will be broken in isolated environments. Replace with a relative link.
  • Line 61, Column 239: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/connectors/webcontentsv2/' 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.

For any questions, please:

@tapanm-MSFT tapanm-MSFT added the powercat-guidance Related to guidance content label Feb 7, 2024
@MicrosoftDocs MicrosoftDocs locked as resolved and limited conversation to collaborators Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants