Issue on page /mastering-plone/features.html - unclear task is solved#958
Issue on page /mastering-plone/features.html - unclear task is solved#958Thanush-03 wants to merge 3 commits intoplone:mainfrom
Conversation
stevepiercy
left a comment
There was a problem hiding this comment.
I don't think these changes are necessary. The UI is similar enough between both frontend and backend.
ksuess
left a comment
There was a problem hiding this comment.
By now there is no "Save and send test e-mail" in the mail control panel of Volto.
But it is in Classic UI: http://localhost:8080/Plone/@@mail-controlpanel
This is why the user should be pointed to http://localhost:8080/Plone/@@mail-controlpanel for verifying she has the entries like being mentioned here and that mails are sent/printed to console.
You may want to use the following:
"With Products.PrintingMailHost, the training set up doesn't send mails, but prints them to the console."
The paragraph on enabling Products.PrintingMailHost in add-on control panel is wrong. It is patching Products.MailHost.MailHost. It is not an add-on to be installed or enabled.
|
@Thanush-03 would you please address the changes that @ksuess requested? Don't keep updating this branch to |
|
ok, i will try to update the requested changes as soon as possible. |
i unable to connect http://localhost:8080/Plone/@@mail-controlpanel this website what should i do now.it showing localhost refused to connect |
mam i update requested changes just check it out. if some changes required just inform me. |
|
@Thanush-03, if you want to learn Plone, please head over to the training https://training.plone.org |
|
i haven't given ai to fix. i only trying to understand the code space and trying to sort it out and as am fresher may be there will be wrong.While you told two changes i thought it as html paragraph changes required i change it with my sentence instead of copying your sentence and also i asked the screenshot what changes required as i didn't understand you not provided. I am sitting from a week to understand the code and the test data base. kindly help me to understand where the changes required and try to provide video if possible. |
|
@Thanush-03 @ksuess would you please connect to resolve what, if anything, needs to be done for this open PR? I'm not clear myself. If nothing needs to be done, then please close it as "won't fix". Thank you! |
|
@Thanush-03 , I suggest you install the training setup. Then you see what I mean by my change requests https://github.com/plone/pull/pullrequestreview-3135560713. I'm not willing to give you screenshots of obvious, decrypt your confused posts and ask what you mean, what you have installed and what you see when you do this and that. |
|
And for a review, please commit your locally made changes. |
|
Superseeding by #967 as you did not commit your local changes. |
What I changed:
Added clear backend navigation steps in docs/mastering-plone/features.md under “Configure a mail server”:
How to reach Site Setup from the frontend.
Direct backend URL (http://localhost:8080/Plone) and where to click (Mail).
Added instructions to enable the add-on:
Go to Add-ons in Site Setup and activate Printing Mail Host (Products.PrintingMailHost).
Kept the training guidance that emails are shown in the console and to use “Save and send test e‑mail” to verify.
Why because
Removes ambiguity for beginners switching from frontend to backend.
Ensures users know where to configure mail and how to make Products.PrintingMailHost effective.
Scope
Only docs/mastering-plone/features.md was edited; no other files required changes.
📚 Documentation preview 📚: https://plone-training--958.org.readthedocs.build/