OfficeDev / office-js

A repo and NPM package for Office.js, corresponding to a copy of what gets published to the official "evergreen" Office.js CDN, at https://appsforoffice.microsoft.com/lib/1/hosted/office.js.
https://learn.microsoft.com/javascript/api/overview
Other
686 stars 95 forks source link

Spam addin event.options is an array in classic outlook and is an object/structure in browsers/new look outlook. #4573

Open glr0221 opened 5 months ago

glr0221 commented 5 months ago

Provide required information needed to triage your issue

The property event.options is of a different type in classic outlook and in browsers/new look outlook.

Your Environment

Expected behavior

I don't know what the expected behavior is. Should event.options be an array? Or should it be an object?

Current behavior

Event.options in spam addin for classic outlook is an array WITH length attribute. Event.options in spam addin for new look outlook (or browser) is a structure WITHOUT length attribute.

Steps to reproduce

  1. Download the sample code in : https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/outlook-spam-reporting
  2. Sideload manifest.xml
  3. Ensure that you have a debugging setup for spam addins in classic outlook.
  4. Create a breakpoint in the part where the value Event.options can be viewed by the debugger.
  5. Start debugging.
  6. When viewing the value of Event.options in classic outlook, it would look like this : spamAddinOptionsIsAnArray
  7. Open outlook in a browser and set a breakpoint where you can check value of event.options.
  8. Start debugging in the browser.
  9. When breakpoint is hit, and debugging paused, check the value of event.options.
  10. You will see that it is not an array contradicting the one in classic outlook. spamAddinOptionsIsNotAnArrayWeb

Link to live example(s)

NA

Provide additional details

NA

Context

Useful logs

Thank you for taking the time to report an issue. Our triage team will respond to you in less than 72 hours. Normally, response time is <10 hours Monday through Friday. We do not triage on weekends.

mobisw-msft commented 5 months ago

Thank you for reporting this issue. The expected behaviour is that for OWA/ New Outlook too the options should be an Array. This has been put in our backlog. Unfortunately, we have no timelines to share at this point.

Internal tracking ID: 4659930