ledger123 / runmyaccounts

SQL-Ledger Customizations by Run My Accounts
GNU General Public License v2.0
16 stars 9 forks source link

[IBP-1033/IBP-829] Create table for emails #560

Open barworobiec opened 2 months ago

barworobiec commented 2 months ago

Hi @ledger123, currently we have a few issues where emails are marked as sent, but they are not visible in our mail provider. Could you please create a new table called 'emails' in the ledger DB, and have there all emails which are sent by SQL Ledger itself, with error description if failed. It will simplify our investigation, as for now the only way to check it, is to check logs in our mail provider.

ledger123 commented 2 months ago

Hi @Worobiec Bartosz @.***> , got it.

I get it done and update.

Regards

On Tue, 10 Sept 2024 at 15:34, Bartosz Worobiec @.***> wrote:

Hi @ledger123 https://github.com/ledger123, currently we have a few issues where emails are marked as sent, but they are not visible in our mail provider. Could you please create a new table called 'emails' in the ledger DB, and have there all emails which are sent by SQL Ledger itself, with error description if failed. It will simplify our investigation, as for now the only way to check it, is to check logs in our mail provider.

— Reply to this email directly, view it on GitHub https://github.com/ledger123/runmyaccounts/issues/560, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADKHPYEJ6XFHK6K54U3JLZV3DLJAVCNFSM6AAAAABN6OEH4KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTMMBWGAZTOMQ . You are receiving this because you were mentioned.Message ID: @.***>

barworobiec commented 2 months ago

Hi @ledger123 any update on that?

ledger123 commented 1 month ago

Hi @barworobiec ,

This is done. Commit is attached to this ticket below.

Please create this table and send emails from sql-ledger and emails will be recorded in this table:

CREATE TABLE email_data ( id SERIAL PRIMARY KEY, notify BOOLEAN, fromname VARCHAR(255), apikey VARCHAR(255), reply_to VARCHAR(255), cc VARCHAR(255), subject VARCHAR(255), charset VARCHAR(50), to_email VARCHAR(255), attachments JSONB, format VARCHAR(50), from_address VARCHAR(255), fileid VARCHAR(255), message TEXT, replyto VARCHAR(255), bcc VARCHAR(255), dbname varchar(50) );

Here is a screen shot of this table with data.

image

ledger123 commented 1 month ago

Hi @barworobiec,

A couple of points:

  1. We cannot determine if the email sent successfully or not once it is handed over to postfix. Though with api/sendinblue we probably can do that with their callback to an endpoint on our server.

  2. You can also configure your postfix to send all outgoing email by adding following line to the config:

always_bcc = bccall@run-my-accounts.com

Regards