Skip to content

Configure option to export portal settings#269

Open
ThibautBorn wants to merge 2 commits intomainfrom
export_root_config
Open

Configure option to export portal settings#269
ThibautBorn wants to merge 2 commits intomainfrom
export_root_config

Conversation

@ThibautBorn
Copy link
Contributor

@ThibautBorn ThibautBorn commented Nov 3, 2025

A fix for issue #270

By adding an option on the relevant export classes (localroles, portlets, default pages), we can prevent the unnecessary export/import of the portal settings.

We set the 'ignore' value to false, so we won't change the current default.

might need to be extended to relations?

@ThibautBorn ThibautBorn marked this pull request as draft November 3, 2025 16:45
@ThibautBorn ThibautBorn changed the title WIP: Configure option to export root settings Configure option to export root settings Nov 3, 2025
@ThibautBorn ThibautBorn marked this pull request as ready for review November 6, 2025 13:43
@ThibautBorn ThibautBorn requested review from gotcha and pbauer November 6, 2025 13:44
@ThibautBorn
Copy link
Contributor Author

@pbauer, Is this something that is generic enough for collective.exportimport?

@gotcha gotcha changed the title Configure option to export root settings Configure option to export portal settings Jan 22, 2026
@gotcha gotcha force-pushed the export_root_config branch 5 times, most recently from ab330d9 to 28f2eb0 Compare January 22, 2026 15:01
context: Some root settings are now exported & imported by default.
Which might cause issues when migrating into an existing site. Allow
users to chose if they need root settings.

For local roles, default pages, portlets
@gotcha gotcha force-pushed the export_root_config branch from 28f2eb0 to a4d93b8 Compare January 22, 2026 15:15
Copy link
Member

@gotcha gotcha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pbauer Can you take a look and merge and release ?

Or give me the rights to release on PyPI ? (userid: gotcha)

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