search

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
670 stars 96 forks source link

OnMessageSend and OnAppointmentSend event based does not work on OWA #4231

Closed avisheknand closed 6 months ago

avisheknand commented 6 months ago

Provide required information needed to triage your issue

Your Environment

Current behavior

The onMessagesend and onAppointmentSend events is not triggering on OWA, New Outlook. However, works fine on Outlook for Desktop (classic) using the js runtime. This was working fine few weeks ago. We have made no code changes to the App, however the app event has stopped working

Steps to reproduce

1.Create a basic onMessageSend and OnAppointmentSend event based addin

  1. Send a email/event from OWA
mmanjaree-msft commented 6 months ago

@avisheknand Could you please share a repro video and OWA Client version as we are unable to repro this issue. Also is this an intermittent issue or does it happen consistently all the time.

avisheknand commented 6 months ago

@mmanjaree-msft ,

We did some further research and think that a recent update to either Office JS or Outlook (OWA and New Outlook) may have caused the problem by changing the way that manifests are processed.

This is what we found:

Previously the events available to New Outlook and OWA (And Classic Outlook) were controlled by the events that were declared in the DesktopFactor section of the manifest.

After the investigation , this seems no longer true. Now it appear that New Outlook and OWA can only see events that are declared in the Mobile factor section of the manifest.

In our case, the events declared for the two factors are different (mainly because many of the events are currently not available on mobile devices).

For us OnMessageSend did not work because it was not declared in the Mobile factor section

We have also recreated it by adding a Mobile factor section to manifest for the sample code (running from GitHub). https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/outlook-verify-sensitivity-label

I have added 2 manifest files, one is the original one and another one with the mobile factor.

From the looks it seems like desktopfactor events are been overwritten or ignored if mobilefactor events are declared when it comes to OWA or New Outlook. manifest-Original.zip manifest-mobile.zip

mmanjaree-msft commented 6 months ago

@avisheknand Thanks for the information. Will investigate and keep you posted here.

mmanjaree-msft commented 6 months ago

@avisheknand Thanks for reporting this issue. We are in process of fixing it. Will keep this post updated. Internal tracking id: 262375

avisheknand commented 6 months ago

@mmanjaree-msft , thanks for looking into this issue. Any updates on the fixes?

ajays-msft commented 6 months ago

@avisheknand - The fix is checked in and rolling out to prod. You should get it by end of this week.

avisheknand commented 6 months ago

@ajays-msft , i have checked and it's working now. Thanks for the fix 🙌👌

ajays-msft commented 6 months ago

closing as the issue is fixed.