Skip to content

Conversation

@ekes
Copy link
Member

@ekes ekes commented Jun 17, 2024

Based on what localgov alert banner does. This adds the scheduled transitions that the permissions are for in #24

However now running ddev drush si -y localgov; ddev drush en -y localgov_workflows; ddev drush en -y localgov_blogs;
I get an error about Services not having installed the workflow yet

  Adding non-existent permissions to a role is not allowed. The incorrect permissions are "add schedule  
  d transitions node localgov_services_landing", "add scheduled transitions node localgov_services_page  
  ", "add scheduled transitions node localgov_services_sublanding", "reschedule scheduled transitions n  
  ode localgov_services_landing", "reschedule scheduled transitions node localgov_services_page", "resc  
  hedule scheduled transitions node localgov_services_sublanding", "view scheduled transitions node loc  
  algov_services_landing", "view scheduled transitions node localgov_services_page", "view scheduled tr  
  ansitions node localgov_services_sublanding".    

@ekes ekes marked this pull request as ready for review June 17, 2024 12:07
@ekes ekes changed the title Start on #24: fails to install because of scheduled transitions permissions Fix #24: fails to install because of scheduled transitions permissions Jun 17, 2024
Copy link
Member

@millnut millnut left a comment

Choose a reason for hiding this comment

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

Looks good to me, just one stray var_dump to remove first

@finnlewis finnlewis merged commit 87288fe into 1.x Jun 18, 2024
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.

4 participants