Skip to content

Improvements to utils.scripts.create_site#130

Merged
ericof merged 1 commit into
mainfrom
issue-441
Jul 6, 2026
Merged

Improvements to utils.scripts.create_site#130
ericof merged 1 commit into
mainfrom
issue-441

Conversation

@ericof

@ericof ericof commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Improvements to utils.scripts.create_site

Implements the two requests from issue #441:

  • package_iface now accepts a list of interfacescreate_site takes either a single browser-layer interface or a sequence of them, normalised internally into a tuple applied to the request.
  • Additional Generic Setup profiles — a new additional_profiles argument installs extra GS profiles on newly created sites (via the new _install_additional_profiles helper).

Also in this PR

  • Refactored the create_site.py script to resolve answers through the shared OPTIONS / environment-variable mechanism instead of a hard-coded ANSWERS dict.
  • Refreshed the test default.json to kitconcept-core defaults.
  • Rewrote the docstrings across utils.scripts in reStructuredText.
  • Added an end-to-end create_site test (TestCreateSite) plus expanded parse_answers parametrisation.

Testing

  • make test274 passed
  • ruff format + ruff check — clean

Extend `utils.scripts.create_site` to accept a list of browser-layer
interfaces via `package_iface` and to install additional Generic Setup
profiles on newly created sites. Refactor the `create_site.py` script to
resolve answers through the shared OPTIONS/env-var mechanism, refresh the
test answers to kitconcept core defaults, and add an end-to-end
`create_site` test. Docstrings across the module were rewritten in
reStructuredText.

Refs: https://gitlab.kitconcept.io/kitconcept/distribution-kitconcept-intranet/-/issues/441
@ericof ericof requested a review from sneridagh July 3, 2026 15:35
@ericof ericof merged commit ddd25bb into main Jul 6, 2026
12 checks passed
@ericof ericof deleted the issue-441 branch July 6, 2026 13:05
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.

2 participants