Skip to content

[Seven] Slots Manager View - #8268

Open
arybakov05 wants to merge 4 commits into
sevenfrom
8264-seven-slots-manager
Open

[Seven] Slots Manager View#8268
arybakov05 wants to merge 4 commits into
sevenfrom
8264-seven-slots-manager

Conversation

@arybakov05

Copy link
Copy Markdown

see #8264

@arybakov05 arybakov05 linked an issue May 21, 2026 that may be closed by this pull request
@davisagli davisagli added this to the Seven milestone May 23, 2026
@@ -0,0 +1,89 @@
import fs from 'fs';

@sneridagh sneridagh May 24, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@arybakov05 Why do you have to do this static analysis of the build? Wouldn't be enough that the route reads the config (config.slots) and builds it from there?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@sneridagh The issue with config.slots is that it only captures the registered slots in a flat hierarchy. The static analysis also captures (empty) SlotRenderers and is able to reconstruct the hierarchy as nested in the implementation.

@sneridagh

Copy link
Copy Markdown
Member

Important

Hello! Thanks for contributing to Plone Aurora! FYI, since June 5th, Plone Aurora lives in its own repository, thus, the seven branch does not accept any merges. If you want to merge your PR, please follow instructions: #8318
Sorry for the inconvenience!

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.

[Aurora] Slots Manager View

3 participants