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

[READ] Step 1: Are you in the right place?

Issues filed here should be about bugs for a specific extension in this repository. If you have a general question, need help debugging, or fall into some other category use one of these other channels:

For general technical questions, post a question on StackOverflow with the firebase tag.
For general Firebase discussion, use the firebase-talk google group.
To file a bug against the Firebase Extensions platform, or for an issue affecting multiple extensions, please reach out to Firebase support directly.

[REQUIRED] Step 2: Describe your configuration

Extension name: firestore-send-email
Extension version: 0.1.32
Configuration values (redact info where appropriate):
- DEFAULT_FROM=kingu.dev support@kingu.dev
- DEFAULT_REPLY_TO=support@kingu.dev
- MAIL_COLLECTION=mails_v1
- SMTP_CONNECTION_URI=smtps://apikey@smtp.sendgrid.net:465
- SMTP_PASSWORD=projects/${param:PROJECT_NUMBER}/secrets/firestore-send-email-SMTP_PASSWORD/versions/latest
- TEMPLATES_COLLECTION=mailTemplates_v1
- TTL_EXPIRE_TYPE=never
- TTL_EXPIRE_VALUE=1

[REQUIRED] Step 3: Describe the problem

After upgrading to 0.1.32, delivery.info.accepted, delivery.info.messageId, delivery.info..response is missing.

Steps to reproduce:

Upgraded to 0.1.32

Expected result

delivery.info should not be missing.

Actual result

v0.1.31 スクリーンショット 2024-03-26 16 06 18

v0.1.32 スクリーンショット 2024-03-26 16 06 42

About this issue

Original URL
State: closed
Created 3 months ago
Comments: 17 (6 by maintainers)

Most upvoted comments

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.

belwalravi on Apr 4, 2024

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

Aconcagua-CTO on Apr 4, 2024

Sorry to hear that. I’m fine with my workaround for now. I don’t use sendgrid templates. And the space in the text field shouldnt cause problems if you revert the change.

New param sounds good.

Regards

On Tue, Mar 26, 2024, 1:14 PM Mais Alheraki @.***> wrote:

@Aconcagua-CTO https://github.com/Aconcagua-CTO I faced some issues usiNewng 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.

— Reply to this email directly, view it on GitHub https://github.com/firebase/extensions/issues/2004#issuecomment-2020861555, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3OX2PRHZJR2R3MZGN7BN3LY2GNFZAVCNFSM6AAAAABFILNJHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQHA3DCNJVGU . You are receiving this because you were mentioned.Message ID: @.***>

Aconcagua-CTO on Mar 26, 2024