Skip to content

Conversation

mariammorgan
Copy link
Contributor

I added more details about the parameter : as we got lots of queries as it is now updated with some suffix for new provisioned domain

We just make it clear for the customer to not change it manual

Reference https://dev.azure.com/ASIM-Security/Messaging%20Protection/_wiki/wikis/Messaging%20Protection%20Wiki/7169/%F0%9F%93%A3Announcements?anchor=**05/01/2025---microsoft-365-dkim-record-format-update-for-new-custom-domains**

I added more details about the parameter <DynamicPartitionCharacter>:  as we got lots of queries as it is now updated with some suffix for new provisioned domain 

We just make it clear for the customer to not change it manual

Reference https://dev.azure.com/ASIM-Security/Messaging%20Protection/_wiki/wikis/Messaging%20Protection%20Wiki/7169/%F0%9F%93%A3Announcements?anchor=**05/01/2025---microsoft-365-dkim-record-format-update-for-new-custom-domains**
Copy link
Contributor

Learn Build status updates of commit 7fa3d5a:

⚠️ Validation status: warnings

File Status Preview URL Details
defender-office-365/email-authentication-dkim-configure.md ⚠️Warning Details

defender-office-365/email-authentication-dkim-configure.md

  • Line 119, Column 71: [Warning: disallowed-html-tag - See documentation] HTML tag 'DynamicPartitionCharacter' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 121, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

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

Learn Build status updates of commit 401a86a:

✅ Validation status: passed

File Status Preview URL Details
defender-office-365/email-authentication-dkim-configure.md ✅Succeeded

For more details, please refer to the build report.

@chrisda
Copy link
Contributor

chrisda commented Jun 19, 2025

@mariammorgan, you need to agree to the Contributor License Agreement before I can merge the pull request.

Copy link
Contributor

Learn Build status updates of commit 983095c:

⚠️ Validation status: warnings

File Status Preview URL Details
defender-office-365/step-by-step-guides/add-advanced-hunting-community-queries.md ⚠️Warning Details
defender-office-365/email-authentication-dkim-configure.md ✅Succeeded

defender-office-365/step-by-step-guides/add-advanced-hunting-community-queries.md

  • Line 50, Column 8: [Warning: sensitive-language-ma - See documentation] Term 'master branch' is sensitive and should not be used in content or code. Usage: 'Copy the master branch only'

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

Learn Build status updates of commit 5421411:

✅ Validation status: passed

File Status Preview URL Details
defender-office-365/email-authentication-dkim-configure.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

Learn Build status updates of commit 73f11db:

✅ Validation status: passed

File Status Preview URL Details
defender-office-365/email-authentication-dkim-configure.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

Learn Build status updates of commit dc0baad:

✅ Validation status: passed

File Status Preview URL Details
defender-office-365/email-authentication-dkim-configure.md ✅Succeeded

For more details, please refer to the build report.

@chrisda
Copy link
Contributor

chrisda commented Jun 25, 2025

Closed and recreated in a different PR in the Private repo.

@chrisda chrisda closed this Jun 25, 2025
learn-build-service-prod bot pushed a commit that referenced this pull request Jun 25, 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.

2 participants