[office-js][office-js-preview] (Outlook) Document additional mail item properties

[samantharamon]

• Use a meaningful title for the pull request. Include the name of the package modified.
• Test the change in your own code. (Compile and run.)
• Add or edit tests to reflect the change.
• Follow the advice from the readme.
• Avoid common mistakes.
• Run pnpm test <package to test>.

If changing an existing definition:

• Provide a URL to documentation or source code which provides context for the suggested changes: None.
• If this PR brings the type definitions up to date with a new version of the JS library, update the version number in the header.

[typescript-bot]

@samantharamon Thank you for submitting this PR!

This is a live comment which I will keep updated.

2 packages in this PR

• office-js-preview — on npm, on unpkg (author is owner)
• office-js — on npm, on unpkg (author is owner)

Code Reviews

This PR can be merged once it's reviewed.

You can test the changes of this PR in the Playground.

Status

• ✅ No merge conflicts
• ✅ Continuous integration tests have passed
• 🕐 Type definition owners or DT maintainers needs to approve changes which affect more than one package

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.

Inactive

This PR has been inactive for 115 days — it is still unreviewed!

---

Diagnostic Information: What the bot saw about this PR

{
  "type": "info",
  "now": "-",
  "pr_number": 67902,
  "author": "samantharamon",
  "headCommitOid": "aa878944f016148b92f868de1d3ed384ebb14523",
  "mergeBaseOid": "9b0f901d1017d4503b49629e672f8560dd88956e",
  "lastPushDate": "2023-12-16T02:51:52.000Z",
  "lastActivityDate": "2024-04-08T19:34:50.000Z",
  "maintainerBlessed": "Waiting for Code Reviews",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Popular",
  "pkgInfo": [
    {
      "name": "office-js-preview",
      "kind": "edit",
      "files": [
        {
          "path": "types/office-js-preview/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "OfficeDev",
        "Rick-Kirkham",
        "AlexJerabek",
        "ElizabethSamuel-MSFT",
        "alison-mk",
        "samantharamon"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "office-js",
      "kind": "edit",
      "files": [
        {
          "path": "types/office-js/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "OfficeDev",
        "Rick-Kirkham",
        "AlexJerabek",
        "ElizabethSamuel-MSFT",
        "alison-mk",
        "samantharamon"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Popular"
    }
  ],
  "reviews": [
    {
      "type": "stale",
      "reviewer": "ndeleuze",
      "date": "2024-04-05T16:47:38.000Z",
      "abbrOid": "d37c895"
    },
    {
      "type": "stale",
      "reviewer": "Rick-Kirkham",
      "date": "2024-02-06T03:32:14.000Z",
      "abbrOid": "a3c33d0"
    },
    {
      "type": "stale",
      "reviewer": "AlexJerabek",
      "date": "2024-01-24T22:52:39.000Z",
      "abbrOid": "a3c33d0"
    }
  ],
  "mainBotCommentID": 1909025401,
  "ciResult": "pass"
}

[typescript-bot]

🔔 @OfficeDev @Rick-Kirkham @AlexJerabek @ElizabethSamuel-MSFT @alison-mk — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

[typescript-bot]

It has been more than two weeks and this PR still has no reviews.

I'll bump it to the DT maintainer queue. Thank you for your patience, @samantharamon.

(Ping @OfficeDev, @Rick-Kirkham, @AlexJerabek, @ElizabethSamuel-MSFT, @alison-mk.)

[AlexJerabek]

Should the '-' at the end of this line be on the next? This seems like a copy/paste problem elsewhere.

[samantharamon]

The colon-dash format was previously agreed upon with editors to emphasize the list of properties of an object since we couldn't add a new line before the properties. I can look into other ways to better represent this in DT.

[AlexJerabek]

Same ":-" comment.

[AlexJerabek]

@samantharamon, approving with suggestions.

[typescript-bot]

@ndeleuze, @Rick-Kirkham, @AlexJerabek Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?