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
685 stars 95 forks source link

ItemSend for shared mailboxes blocking emails from being sent #3527

Open elemen7s opened 1 year ago

elemen7s commented 1 year ago

When sending an email from a shared mailbox, the email is not sent and is blocked because the event ItemSend is still being hit even though SupportsSharedFolders is not set in the manifest and the default should be false.

The event.completed method below is getting ignored too on ItemSend. event.completed({ allowEvent: true });

Office.context.mailbox.initialData contains an error code of 9020 and permissionLevel of 0 as below:

The same addin works fine for a user mailbox.

Your Environment

Expected behavior

Addin events should not be hit if SupportsSharedFolders is not specified and set to true in the manifest

Current behavior

Shared mailboxes are blocked from sending any emails

Steps to reproduce

Create an addin with ItemSend event which does nothing except to call event.completed({ allowEvent: true }) Send an email from a shared mailbox

exextoc commented 1 year ago

Hey @elemen7s

It's not a repro for us. Can you please share a video or a sample add-in which can help us investigate it further ?

elemen7s commented 1 year ago

Uploaded here - https://wetransfer.com/downloads/25c01a6d6c04df5961a3cde7894d4bc620230721113828/52cf66

exextoc commented 1 year ago

Hi @elemen7s, thanks for sharing the video. From the video it seems like the email is being sent from the context of a group and not a shared mailbox as specified in the title of this issue, Could you please confirm?

elemen7s commented 1 year ago

Yes that is correct. At the time of opening this issue the same exact issue was happening when sending a mail using the 'Send email' button marked in below screenshot, but it seems like this particular case has been fixed.

Only the issue shown in the video remains.

image

exextoc commented 1 year ago

Thanks for confirming. We are able to repro this issue and it has been put into our backlog. We unfortunately have no timelines to share at this point. Thanks!

Internal tracking id: 146213