Skip to content

Conversation

acsr
Copy link
Contributor

@acsr acsr commented Feb 6, 2025

How to master running local zope commands for debugging inside a Docker container.

Foot into the door from an answer to a ticket. To be enhanced as step 2…


πŸ“š Documentation preview πŸ“š: https://plone-training--923.org.readthedocs.build/

…local zope commands for debugging

Foot into the door from an answer to a ticket. To be enhanced as step 2…
@stevepiercy
Copy link
Contributor

@acsr if you have the opportunity to work with the trainers, would you please update this PR for accuracy and completeness? It seems to be in a draft state to me. It also has merge conflicts that you need to resolve. Please let me know. Thank you!

@acsr
Copy link
Contributor Author

acsr commented Sep 9, 2025

@stevepiercy I have looked in to the state and cannot get at once the point how and why the merge conflict is displayed here in github. (I followed the web editor link above https://github.com/plone/training/pull/923/conflicts).

Due to personal issues and no time I cannot follow the sprint in the next days as I would like. I confirmed the general wording of the purpose already. Lets see.

@acsr
Copy link
Contributor Author

acsr commented Sep 9, 2025

I have reread the related issue plone/plone.exportimport#51 and the remarks by @yurj and I am not sure if I get the best approach to make serious suggestions to others in a generic way. I know that @fredvd suggested the use of firing up a dedicated container for the relstorage to filestorage conversion process. He wrapped this up during the Plone Tagung but do not see my skilllevel high enough to finalize the suggestion here.

For me it was enough to get a classic Zope related command to work inside the docker container and recognizing the way to prefix the command with a call of docker-entrypoint.sh to set the environment. I am not sure if this is the right approach and its implications and side effects.

@acsr
Copy link
Contributor Author

acsr commented Sep 9, 2025

@stevepiercy
Copy link
Contributor

@acsr scroll to the end of the pull request. Observe how to resolve merge conflicts:

Screenshot 2025-09-09 at 10 40 20 AM

@acsr acsr marked this pull request as draft September 17, 2025 13:06
acsr and others added 19 commits September 17, 2025 17:07
Teaching implies a in my opinion outdated understanding of top down education. Some thing they cannot teach. Some hate teachers. The actual content of that pages reflect a more modern  aproach and are more inviting. This should be also present in the wording of the TOC and Headlines.

The content can also be improved to a more encouraging onboarding of trainers beyond it is already trying tis.
Corrected the component spelling while exporting it .
Component was PloneReleaseView , and while exporting it was written as PlonereleaseView.

I simply corrected the spelling.
To save user's time from debugging small errors .
…ation part of Volto-hands-on training

In the internationalisation part of the Volto hands on training,we have to first translate the Plone release content type of our site. 
The file to be updated is PloneReleaseView.jsx , but the training documentation says the file to be updated is plonerelease.jsx , which we never created in past.

I corrected the file which should be updated-- (PloneReleaseView.jsx ,and not the non existent plonerelease.jsx)

This can cause confusion to a new developer, and he might create a plonerelease.jsx which technically was PloneReleaseView.jsx .
To load an add-on, the project needs to specify the add-on in the addons key of package.json of main project.
In the training , it was written project.json, I corrected it to package.json.
- Add genindex to toctree and home page
- Sort conf.py to align with Documentation for easier future updates
- Fix syntax errors
- Purge unneeded JavaScript, since it was fixed upstream
aboycalledhero and others added 21 commits September 17, 2025 17:07
Changed title capitalization
- Add `drag.svg` icon
The title needs a clarification and we need a second topic pointing to the latest available training / docs fort Plone 6 Classic / similar to Plone 5
I believe this should be index.js not config.js.
…local zope commands for debugging

Foot into the door from an answer to a ticket. To be enhanced as step 2…
@acsr acsr marked this pull request as ready for review September 17, 2025 15:28
@acsr
Copy link
Contributor Author

acsr commented Sep 17, 2025

As @stevepiercy suggests I would like to have a review by a trainer experienced with docker containers for my change.

Priority Reviewers

The trainings for Installing and Deploying Plone during 2025.ploneconf.org are held by @ericof & @fredvd. Maybe you look first.

The approaches I wrote down came from input by @davisagli and @fredvd. @pbauer was relatively new to the deployment stuff when we fixed our building of images for ARM and troubleshoot the logs. @ksuess if you are into this you can also review this and tenor and scope.

Finally having these tipps after heavily missing them was helpful for me. It is difficult to discover and filter for Plone as newbies with more complex docker setups. Actually the swarm approach is a good entry point when you get a grip at it.

@stevepiercy stevepiercy moved this from New to Todo in Plone Documentation Sep 17, 2025
@stevepiercy stevepiercy moved this from Todo to In Progress in Plone Documentation Sep 17, 2025
@stevepiercy stevepiercy requested a review from fredvd September 17, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

7 participants