Skip to content

3953 move plone app workflow to plone app layout#390

Open
MrTango wants to merge 10 commits intomasterfrom
3953-move-plone-app-workflow-to-plone-app-layout
Open

3953 move plone app workflow to plone app layout#390
MrTango wants to merge 10 commits intomasterfrom
3953-move-plone-app-workflow-to-plone-app-layout

Conversation

@MrTango
Copy link
Copy Markdown
Contributor

@MrTango MrTango commented Feb 13, 2025

No description provided.

Copy link
Copy Markdown
Member

@petschki petschki left a comment

Choose a reason for hiding this comment

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

news entry missing, otherwise LGTM

Comment thread plone/app/layout/profiles/default/browserlayer.xml Outdated
Comment thread plone/app/layout/profiles/uninstall/browserlayer.xml Outdated
Comment thread plone/app/layout/views/configure.zcml Outdated
@MrTango
Copy link
Copy Markdown
Contributor Author

MrTango commented Feb 13, 2025

i added a separate PR #391 for the browserlayer

@MrTango MrTango force-pushed the 3953-move-plone-app-workflow-to-plone-app-layout branch from c7d575b to c4fa40c Compare February 13, 2025 14:43
Comment on lines +3 to +4
<layer interface="plone.app.layout.interfaces.IPloneAppLayoutLayer"
name="plone.app.layout"
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.

Suggested change
<layer interface="plone.app.layout.interfaces.IPloneAppLayoutLayer"
name="plone.app.layout"
<layer name="plone.app.layout"


<include
package="plone.app.workflow"
file="permissions.zcml"
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.

Q: should all the package be included?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants