plone / bobtemplates.plone

Python Code Templates for Plone Projects with mr.bob
https://pypi.org/project/bobtemplates.plone/
GNU General Public License v2.0
24 stars 31 forks source link

add a starter template #526

Closed erral closed 6 months ago

erral commented 1 year ago

With this starter template we can initialize the Plone site with some sane defaults. This sub-template will be used by cookiecutter-plone-starter

Right now it only sets the site title and configures the default languages.

See https://github.com/collective/cookiecutter-plone-starter/pull/25 for reference.

erral commented 1 year ago

@ericof @jensens if you think that more things should be added to the default template when creating a Plone site with cookiecutter-plone-starter, we will add them here.

jensens commented 1 year ago

I have no strong opinion here. Less is more IMO, so this looks fine.

MrTango commented 1 year ago

If we have this in plonecli/bobtemplates.plone we should add docs and describe better what it does. The name starter is rather generic. It might be more clear when you use it from the starter template, but it could also be used inside plonecli directly. Another option would be, that we deliver this sub-template via a new bobtemplates package which also does the registration for plonecli. That would be the better option, if this template is only useful for the starter-template. Not sure if thats the case.

erral commented 1 year ago

I have renamed the template to be plone_site_initialization I think the name is better that the starter one. We may add in the future more configuration items here.

The aim of this subtemplate is to be able to initialize a Plone site using an addon, this way, when installed, the addon can set the site title, available langues, or whatever. It's a kind of "policy product configuration", but I have never liked the "policy" name for this :/

MrTango commented 1 year ago

@erral i mannaged to finally fix the build. But i have a wish regarding the naming. I like the plone_site_initialization, it says what it does. But so far this is only the alias in plonecli. We should rename the whole template as follows:

folder: /bobtemplate/site_initialization global name in setup: plone_site_initialization alias in plonecli: site_initialization

this way it's all clean from the start ;)

erral commented 1 year ago

I have renamed everything to site_initialization :)

erral commented 1 year ago

The strange strings were removed.

MrTango commented 1 year ago

the test setup, is broken here. I tried to fix it, but so far we still have some linting issues inside the generated addon tests. I'm running out of time for the moment, this needs more investigation.

erral commented 1 year ago

I will try to take care of this during the Conference sprint

erral commented 7 months ago

@MrTango I finally removed the additions I made to include Python 3.11 in the test setup and the PR is green.

Including Python 3.11 should be a PR on itself if wanted.