"reply-to" instead of "from" about kirby-uniform HOT 4 CLOSED

This is working better -- with a 'replyTo' line added into the params array right after the "from" line.

Mail is now seems to be being delivered reliably with this form -- including gmail from-addresses. And can still reply to the sender by clicking "reply". However, I'm not getting the 'name' to show up in either of these for some reason. I'm not super familiar with Kirby, so I'm not 100% sure how this is working. That would be preferable to get the send name to show in the "from" line with the host domain. I just hardcoded the hosting domain for a quick fix, but it would be best to make that dynamic obviously -- one of the kirby url functions maybe with preg replace or something?

'from'        =>   a::get($this->data, 'name', '') . ' <[email protected]>',
'replyTo'   =>   a::get($this->data, 'name', '') . ' <' . a::get($this->data, '_from') . '>',

Thanks for your great work on this plugin. I realize I may be missing something that caused me to have this problem in the first place -- and you've dealt with it in some way that I don't see here.

from kirby-uniform.

I totally agree, replyTo would be a better field for the submitter's email address. I'd prefer not to generate the from address directly from the domain name since the user should decide which of their email adresses they want to take. It can be an additional (required) argument in the email array.

As to why the name does not appear I found that it is intentionally removed by the Email class. I guess we could open an issue at the toolkit repo and ask if this can be fixed or we could write our own email service that adds the name to the header again.

from kirby-uniform.

Opened a new issure for the names.

from kirby-uniform.

Thanks, Martin. This looks good to me -- handling the sender in the setup array to give users some flexibility.

This plugin is looking really good, by the way. Glad you renamed it in v.2, because it's potentially a lot more than a "contact form." :)

from kirby-uniform.