-
Notifications
You must be signed in to change notification settings - Fork 6
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
Workflows permissions and transitions #99
Conversation
…uthor, contributor and editor roles using the Role Delegation module (#92)
The functionality of the work hear has been tested and reviewed on #92 so this is ready for a code review; which is mainly tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy with this. I've tested it locally and read the config and code.
Would be good to get an OK from @andybroomfield or @Adnan-cds - based on the discussion on #111 we plan to get this work in first, then split off the localgov_workflows as a separate module.
$en_node = $node_storage->loadRevision($node_storage->getLatestRevisionId($en_node->id())); | ||
$cy_translation = $en_node->getTranslation('cy'); | ||
$this->assertEquals('archived', $en_node->moderation_state->value); | ||
$this->assertEquals('published', $cy_translation->moderation_state->value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this testing of translations @stephen-cox, protecting the future!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok as per #111 this is split out when localgov_workflow is made its own module.
* Added contributor role and updated workflow permissions (localgovdrupal/localgov_core#92) * Added User Manager role with permissions to create users and assign author, contributor and editor roles using the Role Delegation module (localgovdrupal/localgov_core#92) * Updated permissions and added tests for workflow transitions and translated content
Adds contributor and user manager roles to localgov_roles and updates default permissions assigned to all roles in localgov_workflows.
Closes #92