🐛 [firestore-send-email] The `delivery.info` entry is missing after upgrading to 0.1.32. about extensions HOT 17 CLOSED

from extensions.

In my case I'm using it on the emailTemplates collection
I added a field on each document
The field name is "text"
The value is " "
Like this

Hope it helps

from extensions.

Thanks @Aconcagua-CTO bro it helped. Mail are getting delivered now. Now we need to add a text field with a space to get your code working.

from extensions.

I can reproduce, looking into it.

from extensions.

Hi guys, not sure if this is related. I upgraded from 0.1.29 to 0.1.32 yesterday, and my mails are failing with
"Error: String expected for text"

I send my messages using html, and always kept text: null.
This was working on 0.1.29, but not on 0.1.32.

I'll keep testing on my side and let you know if I find anything else.

Regards

from extensions.

@Aconcagua-CTO are you using sendgrid?

from extensions.

Yes

from extensions.

Tried with the following and it was sent successfully (using SendGrid)

from extensions.

It seems that when using https://www.npmjs.com/package/nodemailer-sendgrid the response doesn't include a delivery info in the response & it uses SendGrid v3 API, which I used in the newest version for SendGrid as it enabled using Dynamic Templates.

from extensions.

In my case, I'm using emailTemplates collection to populate my emails.
I just confirmed that the template now needs non-empty text field to work.
On production, extension is 0.1.29 and text field is not in the emailTemplate collection, emails are sent without problem.

On 0.1.32 I tested with a null text field

It failed
Tested an empty string and it failed

Now I'm testing a single space and it's working again

Seems we have separate issues :D

from extensions.

@Aconcagua-CTO interesting, might also be related to the last update to SendGrid, checking now.

from extensions.

@Aconcagua-CTO I faced some issues using the templates collection as well, mostly caused by the new SendGrid transporter. We can get back the old transporter which appears to use SendGrid v2, but this will remove the support for Dynamic Templates. Thinking of introducing a new param for users who would like to use v3.

from extensions.

Hi @Aconcagua-CTO @pr-Mais @KoheiKanagu ,
I am also facing a similar kind of issue, after upgrading to the latest version of the extension I am not able to send mails, while using collection templates but with the previous version It was working. Can someone assist with this?

from extensions.

Hi @belwalravi did you try adding a field "text" with a single space? Look at my past message.

from extensions.

Yes @Aconcagua-CTO I read those comments. Could be please let me know how will the object be looking in my case with text. I mean where should I place this text.

from extensions.

Hi @Aconcagua-CTO This is how my template collection doc is. If you are talking about this? Do you mean I should keep the text field here?

from extensions.

Yes, that was the collection I was referring to.
Glad to help!

from extensions.