Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reply-To option #60

Closed
ghost opened this issue May 24, 2017 · 9 comments
Closed

Reply-To option #60

ghost opened this issue May 24, 2017 · 9 comments

Comments

@ghost
Copy link

ghost commented May 24, 2017

Hello,

It's very useful to be able to add a Reply-To option in a form to allow replying to your visitors using the reply function of your webmail.

Is it possible to add an option the plugin component to be able to define the Reply-To option, you should be able to target a field of the form cause generally, you have to reply to the email address filled in the form.

Thanks

@skydiver
Copy link
Owner

Hello @Alex360hd.

Are you referring to the notification or auto-response email?

@ghost
Copy link
Author

ghost commented May 29, 2017

I'm referring to the notification system. I don't want to send an auto-response email, but I want to be able to reply my customers by using the reply option of my webmail. Actually, the sender is the website, that's why we need a reply-to option to reply to customer directly. That option should allow form token as the reply-to value is generally the email filled by the customer in the same form, so we need a way to give to the component a variable corresponding to the email of the customer.

@ghost
Copy link
Author

ghost commented Jun 12, 2017

Hello,

I added the feature, but I'm new to git and I don't know how to create a pull request yet, so I will post the solution "the old way" !

Here is my SharedProperties.php that add the option in the component and the SendMail.php that take care of this new option when sending a notification.

If somebody can easily create a patch or something easy for the author to integrate, I promise to learn how to make a proper patch later, but no time ATM ^^

@skydiver
Copy link
Owner

Thanks for your code @Alex360hd

This is ready on v1.4.1

I will wait a few days before pushing to October market, meanwhile you can test with the master branch

@ghost
Copy link
Author

ghost commented Jun 19, 2017

Thanks, seems to work well.

@worldofsmarty
Copy link

Well when can we expect it to be pushed to the october market place...!!!

@ghost
Copy link
Author

ghost commented Jun 27, 2017

I recently saw the changelog when I updated my october install, I think it's already implemented.

@worldofsmarty
Copy link

worldofsmarty commented Jun 27, 2017

Thank you for the same. But in that field what to add suppose for example we have email as an id in the form filled. Then We need to write just "email" in the reply to field. M i correct.

@ghost
Copy link
Author

ghost commented Nov 20, 2017

Yes, you have to fill the "name" attribute of your field. Some example :

<input type="email" name="email" />

The value to fill in the component settings for the reply-to option is : email

<input type="email" name="my_email" />

The value should be : my_email

etc..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants