Skip to content

Conversation

@PramodKumarHK89
Copy link

Added note about .eml file support and required flag.

Added note about .eml file support and required flag.
@prmerger-automator
Copy link
Contributor

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

Copy link

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 the documentation for email file handling in the MIP SDK to clarify the requirements for working with .eml files. The changes emphasize that a different configuration flag must be used when handling .eml files compared to .msg files.

Key Changes:

  • Expanded the existing note about .eml file support to specify the required enable_eml_file_type flag
  • Added a code comment example demonstrating how to set the enable_eml_file_type flag

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


> [!NOTE]
> As of MIP SDK 1.17, .eml files are supported. Handling is similar to .msg for labeling/protection scenarios and to .rpmsg for inspection scenarios, depending on the container format.
> As of MIP SDK 1.17, .eml files are supported. Handling is similar to .msg for labeling/protection scenarios and to .rpmsg for inspection scenarios, depending on the container format. When working with .eml files, you must set the *enable_eml_file_type* flag instead of *enable_msg_file_type*. For details on how to set this flag, please refer to the [Labeling of MSG Files](#labeling-of-msg-files) section.
Copy link

Copilot AI Oct 17, 2025

Choose a reason for hiding this comment

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

[nitpick] The reference to 'Labeling of MSG Files' section may be confusing since the section primarily demonstrates MSG file handling. Consider either updating the section title to be more inclusive (e.g., 'Labeling of MSG and EML Files') or creating a dedicated section for EML file configuration to improve clarity.

Copilot uses AI. Check for mistakes.
Copy link
Author

Choose a reason for hiding this comment

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

Made changes as per the recommendations

@v-dirichards
Copy link
Contributor

@msmbaldwin

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Oct 17, 2025
Added support details for .eml files in MIP SDK 1.17.
@v-dirichards
Copy link
Contributor

@msmbaldwin Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge? Thanks!

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