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

Standard "editor" group is "broken" with this module #229

Open
boutell opened this issue Feb 15, 2019 · 3 comments
Open

Standard "editor" group is "broken" with this module #229

boutell opened this issue Feb 15, 2019 · 3 comments
Assignees
Labels

Comments

@boutell
Copy link
Contributor

boutell commented Feb 15, 2019

You can't edit anything because you have no locale permissions, but it's neither documented nor supported to set them via the groups key of apostrophe-users.

Best fix is probably to just automatically kill that key in the presence of workflow. If you want workflow, you should be ready for editing groups and setting up their permissions properly.

Better fix is killing the feature but that's 3.0. (:

@boutell
Copy link
Contributor Author

boutell commented Feb 18, 2019

Maybe we just don't bake the feature into the open source CLI boilerplates. Then it can still exist for agency specific boilerplate if the "end users just make a mess with the groups feature" use case is still of concern.

@stale
Copy link

stale bot commented Jun 19, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 19, 2020
@abea abea added the bug label Jun 22, 2020
@stale stale bot removed the stale label Jun 22, 2020
@abea abea self-assigned this Jun 22, 2020
@abea
Copy link
Contributor

abea commented Jun 22, 2020

We can document this and call that enough for 2.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants