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

Update workbench_moderation, drafty, and dkan_sitewide_panels #2656

Merged
merged 2 commits into from
Sep 12, 2018
Merged

Update workbench_moderation, drafty, and dkan_sitewide_panels #2656

merged 2 commits into from
Sep 12, 2018

Conversation

janette
Copy link
Member

@janette janette commented Sep 10, 2018

connects #2578
Fix for 'View Draft' tab

Updates module versions and changes to panel features. Because we're using panels to display the dataset and resource pages, we need to use "node_content" to place the content in the panel rather than "entity_view," which will not allow us to see the draft version.

How to reproduce

  1. Enable dkan_workflow
  2. Go to an existing dataset, edit it, make changes and save as draft
  3. Click the 'View Draft' tab
  4. Confirm that you are viewing the published version rather than the draft

QA Steps

  • Enable dkan_workflow
  • Go to an existing dataset, edit it, make changes and save as draft
  • Click the 'View Draft' tab
  • Confirm that you are viewing the draft version
  • Repeat steps on a resource

Reminders

  • There is test for the issue.
  • Coding standards checked.
  • Review docs.getdkan.com (or in /docs) to see if it still covers the scope of the PR and update if needed.

@dafeder dafeder merged commit f0296fe into GetDKAN:7.x-1.x Sep 12, 2018
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Update workbench_moderation, drafty, and dkan_sitewide_panels

* Update behat.yml
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Update workbench_moderation, drafty, and dkan_sitewide_panels

* Update behat.yml
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.

2 participants