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

Added send test email option #34

Conversation

Schmidie64
Copy link

@Schmidie64 Schmidie64 commented Sep 6, 2019

Pull Request for Issue joomla#22126 .

Summary of Changes

Added a send test-mail button for @Hackwar PR.
This option allows you to send a test-email with a specific mail-template chosen to every valid email-address.

Testing Instructions

Send_Test_Mail_Button

  1. Go to your Joomla adminsite
  2. Change your url to http://{sitename}/index.php?option=com_mails&view=templates
  3. In the left upper corner you can find the "Send Test E-Mail Button"
  4. Now a modal windows will be open
  5. Now you can enter the email-address where the email will be send to
  6. Choose a template for the test mail
  7. Press the "send test e-mail button"
  8. Now you should received a email with the chosen template

@Hackwar
Copy link
Owner

Hackwar commented Sep 6, 2019

How does that fill the data for the template?

@Schmidie64
Copy link
Author

Currently it use hard coded values. This ist an Open question how to Deal with the dynamic value. Maybee @wilsonge can answer These question.

@wilsonge
Copy link

wilsonge commented Sep 8, 2019

I honestly have no clue how to deal with template data :) You can do plugin events I guess but it kinda sucks because for components sending emails you then mandate plugins per component to send test emails. I also considered in the mail template table storing the extension and going through the bootComponent

@Schmidie64
Copy link
Author

What do you think @Hackwar?

…are any Placeholders in the selected template, they will be replaced, if there is a language override (Administrator-site).

Documentation change required!
@Schmidie64
Copy link
Author

For now i implemented a replacement for the placeholders in the test-email. You can change the placeholder-value via an language override. If there is a final decision on how to deal with this feature, then we can consult again about this function.

@roland-d

@Schmidie64 Schmidie64 marked this pull request as ready for review September 10, 2019 08:49
Hackwar pushed a commit that referenced this pull request Oct 19, 2019
Fix schema updates for consistent use of null values in module table and correct names quoting for PostgreSQL.
@Hackwar Hackwar closed this May 14, 2021
@Hackwar Hackwar deleted the branch Hackwar:j4mailtemplates May 14, 2021 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants