starter icon indicating copy to clipboard operation
starter copied to clipboard

Change space root node to use sling:Folder or sling:OrderedFolder.

Open wildone opened this issue 1 year ago • 4 comments

Is your feature request related to a problem? Please describe. Right now the space node uses a nt:Folder and this does not allow any more content be stored under this node. Can this be changed to use sling:Folder or even better sling:OrderedFolder? This would have much more benefits long term.

wildone avatar May 16 '23 15:05 wildone

Hi, can you provide some more details, what do you mean by space node? If I checked correctly it uses ws:PagesSpace, but maybe I'm checking the wrong node?

paweljozwicki avatar May 19 '23 11:05 paweljozwicki

This is for the /content/<space> folder as you cant add jcr:content node under it, its seems that its an nt:Folder as a base type.

wildone avatar May 22 '23 12:05 wildone

So, we use ws:PagesSpace there and I think we won't decide to change that, right now.

This is our dedicated node type and it only allows adding what we have designed for. We don't want to allow adding anything for which we haven't provided support.

But I think you can always use some mixin type if you want to add something.

paweljozwicki avatar May 26 '23 10:05 paweljozwicki

Right now I need to add space level Config, so I had to create a work around. Keeping space level Config in one place sticks to sling principles.

Also just for the spirit of sling and jcr it's just make sense to have everything writable.

It won't impact any of you features and it will make the adoption more predictable as it will avoid needing to make side patterns.

You've already made a best decision to make space node which allows to store site specific content without making some remote locations like in AEM, make a space very portable. This will essentially fully reinforce that in theory and in practice.

It can still be custom node just could inherit from ordered folder to allow other nodes and ordering.

wildone avatar Jul 13 '23 00:07 wildone