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

Site Administrator cannot view pending comments #199

Closed
mauritsvanrees opened this issue May 10, 2022 · 0 comments
Closed

Site Administrator cannot view pending comments #199

mauritsvanrees opened this issue May 10, 2022 · 0 comments
Assignees

Comments

@mauritsvanrees
Copy link
Member

On the @@moderate-comments page, Site Administrators cannot view pending comments. You currently need the Manager, Owner, or Reviewer role. We should add Site Administrator here.

Seen when checking collective.portlet.discussion on Plone 6. Plone 5.2 should have the same problem.

@mauritsvanrees mauritsvanrees self-assigned this May 10, 2022
mauritsvanrees added a commit that referenced this issue May 10, 2022
They were missing permissions on pending comments.
Fixes #199
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue May 10, 2022
Branch: refs/heads/3.4.x
Date: 2022-05-10T21:46:35+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.app.discussion@4acadac

Grant Site Administrators the same workflow permissions as Managers.

They were missing permissions on pending comments.
Fixes plone/plone.app.discussion#199

Files changed:
A news/199.bugfix
M plone/app/discussion/profiles/default/metadata.xml
M plone/app/discussion/profiles/default/workflows/comment_review_workflow/definition.xml
M plone/app/discussion/upgrades.zcml
Repository: plone.app.discussion

Branch: refs/heads/3.4.x
Date: 2022-05-10T23:00:32+02:00
Author: Jens W. Klein (jensens) <jk@kleinundpartner.at>
Commit: plone/plone.app.discussion@2c450be

Merge pull request #201 from plone/maurits-site-administrators-52

Grant Site Administrators same workflow permissions as Managers [5.2]

Files changed:
A news/199.bugfix
M plone/app/discussion/profiles/default/metadata.xml
M plone/app/discussion/profiles/default/workflows/comment_review_workflow/definition.xml
M plone/app/discussion/upgrades.zcml
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue May 10, 2022
Branch: refs/heads/master
Date: 2022-05-10T21:32:46+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.app.discussion@245a2d9

Grant Site Administrators the same workflow permissions as Managers.

They were missing permissions on pending comments.
Fixes plone/plone.app.discussion#199

Files changed:
A news/199.bugfix
M plone/app/discussion/profiles/default/metadata.xml
M plone/app/discussion/profiles/default/workflows/comment_review_workflow/definition.xml
M plone/app/discussion/upgrades.zcml
Repository: plone.app.discussion

Branch: refs/heads/master
Date: 2022-05-10T21:36:50+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.app.discussion@1ea81e4

Increase metadata version to 2000.

We need to leave some room for another upgrade step in Plone 5.2.

Files changed:
M plone/app/discussion/profiles/default/metadata.xml
M plone/app/discussion/upgrades.zcml
Repository: plone.app.discussion

Branch: refs/heads/master
Date: 2022-05-11T01:51:34+02:00
Author: Jens W. Klein (jensens) <jk@kleinundpartner.at>
Commit: plone/plone.app.discussion@0439e9e

Merge pull request #200 from plone/maurits-site-administrators-master

Grant Site Administrators same workflow permissions as Managers [master]

Files changed:
A news/199.bugfix
M plone/app/discussion/profiles/default/metadata.xml
M plone/app/discussion/profiles/default/workflows/comment_review_workflow/definition.xml
M plone/app/discussion/upgrades.zcml
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

No branches or pull requests

1 participant