Skip to content

Conversation

@huntmj01
Copy link
Contributor

@huntmj01 huntmj01 commented Mar 2, 2022

I was receiving a Microsoft Graph API error when trying to use FluentEmail.Graph to send emails with attachments over 3MB. So, I created a fork of NatchEurope/FluentEmail.Graph that modifies the GraphSender to use an upload session for sending emails with attachments that are 3MB or larger.

Unfortunately, the Microsoft Graph API Send method does not have a SaveSentItems argument like the SendMail method does, so I had to remove the option to disable saving sent items. See Link 1, Link 2, and Link 3.

You probably won't want to merge this into your project since you will lose the SaveSentItems argument. But you can use/reference my code if you do decide to modify the project to use upload sessions. Or at the very least others can use my fork if they need to send emails with larger attachments.

I have opened issue #84

@Edwin2020
Copy link

Hi, May I know, when will this PR get approved and a new package be available ?

@DavidDeSloovere
Copy link
Contributor

Hi @Edwin2020 ,
Merging this would remove an existing feature "SaveSentItems"). So not an option for me.
Refactoring would allow both the original and the new 'upload session' without breaking existing implementations.
We need to implementations of the GraphSender.cs class. Is this something you want to take on?

@DavidDeSloovere DavidDeSloovere merged commit ec75674 into alistar-dev:main Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants