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

Final Workflow content improvements #2606

Merged
merged 6 commits into from
Jul 31, 2018
Merged

Final Workflow content improvements #2606

merged 6 commits into from
Jul 31, 2018

Conversation

stefaniefgray
Copy link

@stefaniefgray stefaniefgray commented Jul 26, 2018

@erogray
Copy link

erogray commented Jul 26, 2018

@janette can you review this when you get a chance?


.. figure:: ../images/workflow/dkan_workflow_screenshot.png

_What you see on My Workbench after login. The Workbench navigation bar contains your content, drafts, and more. The "Create Content" menu features a list of content types you can create.
The above image displays you see on My Workbench after login. The Workbench navigation bar contains your content, drafts, and more. The "Create Content" menu features a list of content types you can create.
Copy link
Member

Choose a reason for hiding this comment

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

The above image displays you see on My Workbench after login.

missing the 'what'


drush en dkan_workflow -y

Enabling DKAN workflow will enable all of the necessary dependencies modules and update site
Copy link
Member

Choose a reason for hiding this comment

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

Enabling DKAN workflow will enable all of the necessary dependencies modules

Sounds weird, how about "Enabling DKAN workflow will automatically enable all other required modules and add the Workflow Supervisor role to all users already assigned the site manager role."

@stefaniefgray
Copy link
Author

These changes look great - let's go ahead!

Copy link

@erogray erogray 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 now

@erogray erogray merged commit a0a5725 into 7.x-1.x Jul 31, 2018
@dafeder dafeder deleted the stefaniefgray-docs3 branch July 31, 2018 16:30
dafeder pushed a commit that referenced this pull request Apr 24, 2020
Final Workflow content improvements
dafeder pushed a commit that referenced this pull request Apr 24, 2020
Final Workflow content improvements
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