[#44002] - [5.3] Send test mail to the logged-in user.
- Fixed in Code Base
- 10 Oct 2024
- Medium
- Build: 5.3-dev
- # 44002
- Diff
- tecpromotion:send-test-mail-to-user
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Summary of Changes
In times of Microsoft 365, transactional mail services, SMTP relays and configurations where there is no easy access to the MailFrom mailbox, it makes sense for the test mail to be sent to the logged-in user.
Usually, the mailbox is accessible and the email settings can be tested with real live data.
Testing Instructions
Log in to the backend with your user and go to Global Configuration.
In the Server tab, complete the email configuration and then click "Send Test Mail".
Actual result BEFORE applying this Pull Request
The test mail is sent to the e-mail address (mailfrom) from the global configuration.
Expected result AFTER applying this Pull Request
The test mail is sent to the logged-in user.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_config |
I have always just changed the from address when testing - I guess this just automates it ;)
I have tested this item ✅ successfully on d2a0792
tested successful, mail was sent to me (the tester / user) instead of smtp account
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44002.
I have tested this item ✅ successfully on d2a0792
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44002.
| Status | Pending | ⇒ | Ready to Commit |
rtc
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44002.
I realy like it, but I think this needs a documentation entry as it is a change in behavior.
I realy like it, but I think this needs a documentation entry as it is a change in behavior.
That was my first thought but the success/failure message is quite clear where the email is sent already
| Labels |
Added:
RTC
PR-5.3-dev
|
||
I really think this should be documented. If done, I'm happy to merge it.
Which sentence would you change here or which one is not correct anymore?
https://docs.joomla.org/How_to_debug_SMTP_mail_in_Joomla_4
I dont see it... @laoneo
With this pr, the mail is sent to the current user and not anymore to the mail address from the config. The sentence is still ok, but the behavior changes and I think this should be mentioned in the documentation of the release. The pr itself is absolutely fine.
To not delay this pr any longer, I'v added the documentation required label, so we will find it again when we do the release notes.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-10-10 12:44:01 |
| Closed_By | ⇒ | laoneo | |
| Labels |
Added:
Documentation Required
|
||
Thanks!
@chmst and @coolcat-creations I have created a PR here for the topic from the DACH channel in Mattermost.
Thanks @bembelimen