Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate ACL page permissions from plugin aclinfo in sidebar #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fschrempf
Copy link
Contributor

@fschrempf fschrempf commented Apr 30, 2021

If the aclinfo plugin is installed a new item for the page permissions will be added to the sidebar menu.

This PR depends on cosmocode/aclinfo#7!

image

If the aclinfo plugin is installed a new item for the page permissions
will be added to the sidebar menu.

Signed-off-by: Frieder Schrempf <[email protected]>
@splitbrain
Copy link
Member

Hmm, I'm not sure why this needs to have any code in the template. Why not simply add the plugin's syntax to the sidebar?

@fschrempf
Copy link
Contributor Author

Hmm, I'm not sure why this needs to have any code in the template. Why not simply add the plugin's syntax to the sidebar?

I just followed the way how the starred plugin is integrated in the sidebar. Only the upper part of the sidebar is generated from wiki syntax and as far as I understand meant to be used for site navigation and links. The lower part (separated by some spacing) is for tools and plugins and this is where the aclinfo nicely fits in. This part is not meant to be generated from any wiki syntax as far as I can see.

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