Customizable Mail Notification

[mztiq]

Hey,

really love pingvin and the feature to send mail notifications for shares.
Any chance we could define that notification message ourselves in a text box or something?

[stonith404]

Good idea👍 This feature is not on the top of priority list, but will get implemented.

[stonith404]

Added in 0616a68 :) This allows you to set the mail message in the config section. This will get released in the next release.

[mztiq]

Wow, that was pretty quick for "not on top of the priority list". Thank you very much 😊.

[mztiq]

Sorry for the late reply, I just had time to test this feature (which works great btw) but I noticed that it would be quite handy to be able to customize the subject of the mail notification too.
In my case all of my users are German and some don't speak English very well so right now I'd have a mix of English subject and German mail content. Any chance of getting a customizable subject too and if so should I create a new issue for that?

[stonith404]

Ah yeah sure, completely forgot that. I've added another config variable cabaee5. I don't release this at the moment because the config table gets now really messy with so many configuration options. I'll officially release this when I've optimized the config page #60 .

But I've made a Docker image for you where I already added the subject config variable: stonith404/pingvin-share:pre-release

[mztiq]

Nice, thank you very much!

[stonith404]

Added in v0.5.0 :)

[mztiq]

Are you sure it's added in v0.5.0?

It looked like this with the stonith404/pingvin-share:pre-release:

And this is from v.0.5.0, the EMAIL_SUBJECT option is missing for me:

[stonith404]

Yeah but the problem is that for some reason the build pipeline takes 1 hour. Could you try to pull again?

[mztiq]

Just pulled again, the EMAIL_SUBJECT option is there now.
Thanks.

[Hukuma1]

Right now the name it uses as sender is Pingvin Share <myemail@mydomain.com>. Could we possibly also get that to be editable? Don't want to confuse my e-mail contacts (as cute as the name of this project is). :D

Also since we already have nice variable like {shareUrl}. Can we please also add {desc} (for the description of share) and {fileList} of the list of files and sizes? And {expires} could show when it would expire. This would bring it on parity with WeTransfer sent e-mails even more I believe. And this way you could personalize the e-mail subject even more by having the {desc} in it, for example.

[stonith404]

@Hukuma1 In case of the name in the email, I've just created an issue that would implement this (#93).
Could you create a separate issue for the extension of the email message variables?