Skip to content

Allow changing the target slot of a viewlet, via configuration, instead of ZCML registration. - #90

Open
MrTango wants to merge 2 commits into
masterfrom
additional-viewlets
Open

MrTango wants to merge 2 commits into
masterfrom
additional-viewlets

Conversation

@MrTango

@MrTango MrTango commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

IAdditionalViewlets subscription adapters for (context, request, view, manager) add (name, viewlet) pairs a manager renders, orders and hides like its registered viewlets. This allows assigning viewlets to managers from configuration instead of ZCML.

IAdditionalViewlets subscription adapters for (context, request, view,
manager) add (name, viewlet) pairs a manager renders, orders and hides
like its registered viewlets. This allows assigning viewlets to managers
from configuration instead of ZCML.
@mister-roboto

Copy link
Copy Markdown

@MrTango thanks for creating this Pull Request and helping to improve Plone!

TL;DR: Finish pushing changes, pass all other checks, then paste a comment:

@jenkins-plone-org please run jobs

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically.

Happy hacking!

@MrTango

MrTango commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

@jenkins-plone-org please run jobs

@MrTango
MrTango requested a review from thet October 2, 2026 20:10
@MrTango MrTango changed the title Allow changing the target slot of a viewlet, via configuration, rather than ZCML registration. Allow changing the target slot of a viewlet, via configuration, instead of ZCML registration. Oct 2, 2026
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