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

Remove workflow emails from dkan_workflow #2870

Merged
merged 4 commits into from
May 15, 2019
Merged

Conversation

dharizza
Copy link
Contributor

@dharizza dharizza commented May 6, 2019

Connects #2857

User story

As a DKAN administrator, I'd like to be able to update the workflow email notification templates for my site without having them revert during the next upgrade, so that I can effectively customize my site's workflow communications.

QA Steps

  • On a fresh install enable the module dkan_workflow, the workbench email templates should be defined as before (look at them in admin/config/workbench/email)
  • Do some changes to the templates.
  • Revert the feature dkan_workflow (you can do it with drush: drush fr dkan_workflow -y).
  • Confirm the templates were not overriden and your changes are still there.

@dharizza dharizza mentioned this pull request May 6, 2019
6 tasks
@dharizza dharizza self-assigned this May 6, 2019
@janette janette merged commit a2c02f0 into 7.x-1.x May 15, 2019
@janette janette deleted the 2857-workflow-emails branch May 15, 2019 20:09
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Import workbench email default templates when dkan_workflow is enabled.

* Document how to reimport workbench email templates from DKAN Workflow.
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Import workbench email default templates when dkan_workflow is enabled.

* Document how to reimport workbench email templates from DKAN Workflow.
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.

None yet

3 participants